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: Allow loading of locales like zh_TW #3895

Merged
merged 2 commits into from
Oct 10, 2023
Merged

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Oct 9, 2023

What

Allow loading of locales like zh_TW

Why

Deactivate the nonExplicitSupportedLngs i18next setting (see https://www.i18next.com/overview/configuration-options#languages-namespaces-resources for more details) that caused our supported languages needed to contain only short codes of the languages to be a valid locale. Therefore locales like zh_TW were considered as not supported and the locale did fall back to en.

References

#3842
#3894

GEA-294

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

Conventional Commits Report

Type Number
Added 1
Bug Fixes 1

🚀 Conventional commits found.

@bjoernricks bjoernricks marked this pull request as ready for review October 9, 2023 10:02
@bjoernricks bjoernricks requested a review from a team as a code owner October 9, 2023 10:02
@bjoernricks bjoernricks enabled auto-merge (rebase) October 9, 2023 11:18
@bjoernricks bjoernricks mentioned this pull request Oct 9, 2023
1 task
Deactivate the `nonExplicitSupportedLngs` i18next setting (see
https://www.i18next.com/overview/configuration-options#languages-namespaces-resources
for more details) that caused our supported languages needed to contain
only short codes of the languages to be a valid locale. Therefore
locales like `zh_TW` were considered as not supported and the locale did
fall back to `en`.
Support simple and traditional Chinese locales for our i18n message
extraction tooling.
@bjoernricks bjoernricks merged commit 59239c1 into main Oct 10, 2023
11 checks passed
@bjoernricks bjoernricks deleted the fix-language-switch branch October 10, 2023 07:18
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