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: use zh-TW and zh-CN over zh-Hant and zh-Hans #377

Merged
merged 1 commit into from
Jan 11, 2018

Conversation

chenglou
Copy link
Contributor

@chenglou chenglou commented Jan 6, 2018

Both crowdin and docsearch i18n configs use TW and CN over Hant and Hans. This should reduce some config confusions.
cc @ericnakagawa. Also, apparently you're working on algolia search i18n?

Both crowdin and docsearch i18n configs use TW and CN over Hant and Hans. This should reduce some config confusions.
cc @ericnakagawa. Also, apparently you're working on algolia search i18n?
@@ -41,8 +41,8 @@ files:
'tr': 'tr'
'uk': 'uk'
'vi': 'vi'
'zh-CN': 'zh-Hans'
'zh-TW': 'zh-Hant'
'zh-CN': 'zh-CN'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hope this and the changes in website/ don't break docusaurus's own site's current crowdin settings. There's no chinese translation for it right now anyway.

@JoelMarcey
Copy link
Contributor

@chenglou Thanks! This seems reasonable to me. We will know after I merge this whether docusaurus.io breaks -- but I don't think it should.

@JoelMarcey JoelMarcey merged commit 00270c2 into facebook:master Jan 11, 2018
@chenglou chenglou deleted the chinese branch January 12, 2018 00:20
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.

3 participants