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 #647 - undefined translation strings #724

Merged

Conversation

navarchus
Copy link
Contributor

@navarchus navarchus commented Aug 21, 2024

Fix for #647

  1. "death.attack.hexcasting.overcast.player" was missing, added missing string in en_us, still needs to be defined in ru_ru and zh_cn. Verified works (1.20.1, forge-47.1.47).

  2. changed stat.hexcasting.media_overcasted to stat.hexcasting.media_overcast in en_us, ru_ru, and zh_cn. Verified works (1.20.1, forge-47.1.47).

  3. was fixed before this PR. Verified works (1.20.1, forge-47.1.47).

@navarchus navarchus changed the title Fix #Fix undefinined translations Fix #647 - undefinined translations Aug 21, 2024
@navarchus navarchus changed the title Fix #647 - undefinined translations Fix #647 - undefinined translation strings Aug 21, 2024
@navarchus navarchus changed the title Fix #647 - undefinined translation strings Fix #647 - undefined translation strings Aug 21, 2024
@object-Object
Copy link
Member

Datagen shouldn't need to be run for this change btw. The build was failing due to an intermittent ForgeGradle issue (which should be mitigated by #696).

@object-Object
Copy link
Member

image
image
image

cool cool - thanks for the PR

@object-Object object-Object merged commit 7ab5289 into FallingColors:main Aug 21, 2024
4 checks passed
@navarchus navarchus deleted the fix-undefined-translation-strings branch August 22, 2024 03:59
@navarchus navarchus restored the fix-undefined-translation-strings branch August 22, 2024 03:59
@navarchus navarchus deleted the fix-undefined-translation-strings branch August 22, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants