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

Tweak QuickStart-Tutorial.zh-CN.md metadata for consistency #1239

Closed
wants to merge 1 commit into from
Closed

Tweak QuickStart-Tutorial.zh-CN.md metadata for consistency #1239

wants to merge 1 commit into from

Conversation

wincent
Copy link
Contributor

@wincent wincent commented Jun 24, 2016

Make this one like our other translations:

  • Use .zh-CN suffix instead of -zh-CN: this means that an appropriately configured web server can choose to serve up the right file based on the Accept-Language header.
  • Use same id as canonical version. We might end up changing our minds about that (see Teach website sidebar to prefer en-US #1238), but in the meantime we should at least be consistent.

I think we likely still have some issues here, so I'm going to keep poking around after this. Basically, I think at this point that we've unbroken our English docs and our Chinese docs are at least consistently defined, but the build process for the translations isn't producing a useful/accessible result.

Make this one like our other translations:

- Use `.zh-CN` suffix instead of `-zh-CN`: this means that an appropriately
  configured web server can choose to serve up the right file based on the
  `Accept-Language` header.
- Use same `id` as canonical version. We might end up changing our minds about
  that (see #1238), but in the meantime we
  should at least be consistent.

I think we likely still have some issues here, so I'm going to keep poking
around after this.
@wincent
Copy link
Contributor Author

wincent commented Jun 24, 2016

@facebook-github-bot import

@ghost ghost added the CLA Signed label Jun 24, 2016
@facebook-github-bot
Copy link
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

@ghost ghost closed this in ee9924c Jun 24, 2016
@wincent wincent deleted the glh/localization-names branch June 24, 2016 23:34
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants