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

🌐 i18n: fix merge translations if structure differs #1069

Merged

Conversation

JGreenlee
Copy link
Collaborator

If the structure is different between lang and fallbackLang such that a key is an object in fallbackLang and a string in lang, we experience an error when we attempt to assign a property on the string. This change will skip filling in if the key is not an object in both lang and fallbackLang, thus preventing the error.

If the structure is different between lang and fallbackLang such that a key is an object in fallbackLang and a string in lang, we experience an error when we attempt to assign fill in the property to the string.
This change will skip filling in if the key is not an object in *both* lang and fallbackLang, thus preventing the error.
@shankari shankari changed the base branch from master to service_rewrite_2023 October 29, 2023 02:24
@shankari
Copy link
Contributor

shankari commented Oct 29, 2023

merging to service rewrite to make the prettier reformat easier
everybody should currently be working on the rewrite anyway, so everybody should be able to get the changes anyway

@shankari shankari merged commit 56014a5 into e-mission:service_rewrite_2023 Oct 29, 2023
4 checks passed
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