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 #5379 #5437

Merged
merged 8 commits into from
Jan 23, 2024
Merged

Conversation

ZZZank
Copy link
Contributor

@ZZZank ZZZank commented Jan 22, 2024

This PR, combined with my previous PRs(like #5435 ), fixes #5379 , with some additions.

This PR enables localizations for:

  • Special custom items
  • Multiblock gamestage error message
  • Custom Amadron trades

and

  • provides corresponding zh_cn localization for new entries
  • use JSON.stringify to generate nbt string, for a better format
  • fixes some wrong translations I made in Fluid registry rework #5435 Switched to Crowdin

@ZZZank
Copy link
Contributor Author

ZZZank commented Jan 22, 2024

I also hava a problem: When will Crowdin project gets update?
I'm asking this because of those translation errors I made in PR4535. They are not severe errors, just missing "bucket" when localizing fluid bucket names, but they're still errors. Translating on Crowdin can largely avoid and qiuckly fix those errors, because there's more participants.

@NielsPilgaard
Copy link
Collaborator

Thanks for the contribution, I'll review within a day or two 👍

I also hava a problem: When will Crowdin project gets update?

I've just synced, so it should be good now - Apparently it got stuck in December 2022 😅

@@ -10,7 +10,13 @@ onEvent('recipes', (event) => {
type: 'ITEM',
id: 'pneumaticcraft:reinforced_chest',
amount: 1,
nbt: `{display:{Name:'[{"text":"Atum Mineral Survey"}]',Lore:['[{"text":"A collection of minerals from Atum.","color":"gold"}]']},BlockEntityTag:{LootTable:"enigmatica:chests/amadron_mineral_survey_atum_combo"}}`
nbt: JSON.stringify({
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice change 👌

@NielsPilgaard NielsPilgaard merged commit 934297b into EnigmaticaModpacks:develop Jan 23, 2024
7 checks passed
@NielsPilgaard
Copy link
Collaborator

Excellent PR as usual, thank you 😊

@ZZZank ZZZank deleted the more_localization branch January 24, 2024 14:29
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