-
Notifications
You must be signed in to change notification settings - Fork 945
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
Review translations configuration and documentation #950
Conversation
lyyder
commented
Nov 13, 2018
•
edited
Loading
edited
- remove unnecessary language configurations
- update translations documentation
9b7e692
to
0e5d893
Compare
docs/translations.md
Outdated
* In [src/app.js](../src/app.js), change the translation imports to point to the correct | ||
`react-intl` locale and the new translations file you created, for example: | ||
|
||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add "js"
docs/translations.md
Outdated
|
||
1. In [src/app.js](../src/app.js), change the translation imports to point to the correct | ||
`react-intl` locale and the new translations file you created. | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add "js"
docs/translations.md
Outdated
in tests. To change the translation file used in tests change the `messages` variable in | ||
[src/util/test-helpers.js](../src/util/test-helpers.js) to match your language in use, for example: | ||
|
||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add "js"
@@ -234,6 +234,7 @@ export class SearchPageComponent extends Component { | |||
onCloseAsModal={() => { | |||
onManageDisableScrolling('SearchPage.map', false); | |||
}} | |||
messages={intl.messages} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add js
to the beginning of the code block: "```js"
Translations are passed as props so the handling for translations in tests is already done in test-helpers.js.
26a5b06
to
501d5b0
Compare