Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

i18n proof of concept changes #128

Closed
snostorm opened this issue Mar 5, 2015 · 3 comments
Closed

i18n proof of concept changes #128

snostorm opened this issue Mar 5, 2015 · 3 comments

Comments

@snostorm
Copy link

snostorm commented Mar 5, 2015

Hi, I just opened a PR which is another step towards trying to improve our i18n workflows. I used the cn localization as one of the examples. nodejs/iojs.org#276

Note: eventually, when this is merged, we will not have duplicate copies of the home.html in each localization (and index.md will be retired.)

I'm interested to hear your feedback or suggestions.

@snostorm snostorm changed the title Proof of concept i18n proof of concept changes Mar 5, 2015
@fengmk2
Copy link
Contributor

fengmk2 commented Mar 6, 2015

@snostorm Look good to me. @xudafeng should also need to know this change.

@xudafeng
Copy link
Contributor

xudafeng commented Mar 6, 2015

Agree with you, this will be a reasonable improvement. @snostorm @fengmk2

@snostorm
Copy link
Author

snostorm commented Mar 6, 2015

Thanks for the feedback. Feel free to +1 the direction over on the main PR -- nodejs/iojs.org#276 -- as well, it'll help keep the discussion centralized.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants