Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

%num1% not replaced in buff description #108

Open
ideka opened this issue May 26, 2024 · 4 comments
Open

%num1% not replaced in buff description #108

ideka opened this issue May 26, 2024 · 4 comments
Labels
API Problem or Feature Request not related to the client lib but the API bug Something isn't working ingame issue An issue that exists in the library but behaves the same in game

Comments

@ideka
Copy link

ideka commented May 26, 2024

All skills in palette 2934 (all enemy skills) reference a buff with this issue. The buff in question is 12862.

I'm guessing it should be changed around here but I'm not sure what the value is supposed to be.

@ideka ideka added the bug Something isn't working label May 26, 2024
@SaculRennorb
Copy link
Member

Well, to be fair those skills don't properly work in game either:
grafik

I can however probably just fix them on the server side, shouldn't even be a huge deal.

I'll keep you updated.

@SaculRennorb SaculRennorb added API Problem or Feature Request not related to the client lib but the API ingame issue An issue that exists in the library but behaves the same in game labels May 26, 2024
@SaculRennorb
Copy link
Member

Ok, in a moment the server will have pulled some new processing logic and i've already added some code to this lib to specifically account for this case (fix/108).

The code is very targeted which i usually try to avoid, which is also the reason why this is on a separate branch for now. You now have a fix for the issue, but im still debating if i should merge the solution int the main project.

Please add information about any similar issues, should you find any in the future.

@ideka
Copy link
Author

ideka commented May 27, 2024

Oh... dunno why I didn't think of creating a chat link to see how it looked ingame. Honestly if it's broken ingame too then I'd probably just leave it.

I will report back if I find other similar issues.

@SaculRennorb
Copy link
Member

Well,,, part of the manifesto is to also fix things that are broken in the actual data, so i don't have any ironing out something like that. Its just I don't like hardcoding any ids, that's why I'm a bit hesitant about the current solution.

btw, this is how our version looks now:
img

interestingly there are some rounding errors on other parts - i need to look into those, but that should be unrelated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Problem or Feature Request not related to the client lib but the API bug Something isn't working ingame issue An issue that exists in the library but behaves the same in game
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants