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

Fix world ratio oversight #37

Merged
merged 15 commits into from
Sep 4, 2021
Merged

Fix world ratio oversight #37

merged 15 commits into from
Sep 4, 2021

Conversation

AtriusX
Copy link
Owner

@AtriusX AtriusX commented Aug 25, 2021

This PR primarily aims to fix an issue with world ratios working primarily off of world names and not dimension type. Both functionalities are useful in different ways, so this updates the system to make use of both approaches. When retrieving a ratio, the ratio will first attempt to retrieve by world name, and then by dimension type if no ratio is set.

An additional side effect of this change is that it's now possible to modify the world ratios file via a new command. This will make configuration of this system much more accessible to admins who do not have file system access. This also results in the inclusion of a new permission: waystones.ratio.

@AtriusX AtriusX added enhancement New feature or request needs localization Requires translation into other languages labels Aug 25, 2021
@AtriusX AtriusX marked this pull request as ready for review September 4, 2021 02:47
NewbieOrange added a commit to NewbieOrange/Waystones that referenced this pull request Sep 4, 2021
This PR is missing translation for `HOW_LOW_CAN_YOU_GO` since it is unused and without context I am not sure how to translate it.
AtriusX added a commit that referenced this pull request Sep 4, 2021
Add Chineses localization for #36 and #37
@AtriusX AtriusX merged commit b8180ad into master Sep 4, 2021
@AtriusX AtriusX deleted the fix/world-ratio branch September 4, 2021 22:38
AtriusX added a commit that referenced this pull request Sep 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs localization Requires translation into other languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant