-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix #21434: Number of guests overflows in objective text #21518
Fix #21434: Number of guests overflows in objective text #21518
Conversation
ae4af45
to
e9977d1
Compare
2b2c91c
to
a56ae7d
Compare
Resolved review comment, rebased to develop to resolve conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks fine haven't tested it though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested it, works correctly. Could you add a changelog entry?
a56ae7d
to
204e23a
Compare
Changelog entry added. |
204e23a
to
ce17952
Compare
ce17952
to
e47400b
Compare
@jan-kelemen Could you also create a PR against the Localisation repository that replaces the formatting token in all other languages? |
@Gymnasiast yup, done OpenRCT2/Localisation#2787 |
OpenRCT2#21518 made some changes to objective formatting, but forgot to update the scenario selector. This PR addresses this, and also moves the code into a function of its own to reduce code duplication and prevent this problem in the future.
Format number of guests in objective as Comma32 instead of Comma16.
Fixes [#21434]