Skip to content

Automatically generate i18n javascript files for react-intl when the serve starts up.#642

Merged
justin808 merged 1 commit intoshakacode:masterfrom
JasonYCHuang:i18n
Jan 12, 2017
Merged

Automatically generate i18n javascript files for react-intl when the serve starts up.#642
justin808 merged 1 commit intoshakacode:masterfrom
JasonYCHuang:i18n

Conversation

@JasonYCHuang
Copy link
Copy Markdown
Contributor

@JasonYCHuang JasonYCHuang commented Dec 10, 2016

This PR is for shakacode/react-webpack-rails-tutorial#340
We need i18n_dir & default_locale in ReactOnRails configuration.


This change is Reviewable

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.04%) to 82.792% when pulling 38e92e2 on JasonYCHuang:i18n into 6370447 on shakacode:master.

@justin808
Copy link
Copy Markdown
Member

we want this pr to include i18n.rake


Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@justin808
Copy link
Copy Markdown
Member

@JasonYCHuang How's this PR going?

@JasonYCHuang
Copy link
Copy Markdown
Contributor Author

@justin808 I am occupied recently, and will finish this during Xmas or New year holidays.

@JasonYCHuang JasonYCHuang force-pushed the i18n branch 3 times, most recently from fdff007 to c367290 Compare December 27, 2016 10:02
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 27, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling c367290 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling c367290 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling c367290 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@justin808
Copy link
Copy Markdown
Member

@JasonYCHuang Looks potentially super amazing! We need updates to the docs and a CHANGELOG.md, as well as tests that demonstrate and verify locals_to_js.rb.

This is really a huge addition to the project! 👏 🎉


Reviewed 2 of 5 files at r2.
Review status: 2 of 5 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@JasonYCHuang
Copy link
Copy Markdown
Contributor Author

No problem, will update it later.


Review status: 2 of 5 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@JasonYCHuang JasonYCHuang changed the title add i18n_dir & default_locale to configuration Automatically generate javascript files for react-intl when the serve starts up. Dec 31, 2016
@JasonYCHuang JasonYCHuang changed the title Automatically generate javascript files for react-intl when the serve starts up. Automatically generate i18n javascript files for react-intl when the serve starts up. Dec 31, 2016
@JasonYCHuang JasonYCHuang force-pushed the i18n branch 2 times, most recently from d2188f1 to 95d8a08 Compare December 31, 2016 13:08
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 95d8a08 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

1 similar comment
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 95d8a08 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 95d8a08 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@JasonYCHuang JasonYCHuang force-pushed the i18n branch 4 times, most recently from ffacf2c to 2870647 Compare December 31, 2016 13:36
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 2870647 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 2870647 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 2870647 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@JasonYCHuang
Copy link
Copy Markdown
Contributor Author

Update Readme, changelog, and add test


Review status: 2 of 8 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 2870647 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2016

Coverage Status

Coverage increased (+0.02%) to 99.316% when pulling 2870647 on JasonYCHuang:i18n into 13cff3f on shakacode:master.

ryanbelke pushed a commit to ryanbelke/rails-react-yourtime that referenced this pull request Jan 4, 2019
joseph0919 pushed a commit to joseph0919/React_Webpack_Rails_Tutorial that referenced this pull request May 3, 2019
jimmydev996 pushed a commit to jimmydev996/React-Webpack-Rails-Tutorial that referenced this pull request Oct 30, 2025
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