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 locale placement #26

Open
wants to merge 1 commit into
base: fabric-1.18.2
Choose a base branch
from

Conversation

Ratismal
Copy link

The mod currently has lang files under the minecraft namespace. This leads to an issue where it wipes out almost all locale strings that were loaded into that namespace, presumably all strings loaded before this mod. Which means that on server side, console outputs get replaced with just their keys, as theres no string to localize to.
2022-11-29_15-14-59_explorer

This PR moves the locale strings under the structures compass namespace, repairing the console. I did this fix for my own sanity and am using my own build on my server, so feel free to accept or ignore.

@Samarium150
Copy link
Owner

Does this bug also affect the client side?

@Ratismal
Copy link
Author

Just server-side, I think. It's pretty obvious if you even just boot the dev server. I'm not entirely sure why it happens on the server - it is pretty weird. I guess they parse differently?

I know that this still happens with current 1.19 fabric as well, as I've run into it before with my own mods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants