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

Switch to crowdin #1171

Merged
merged 2 commits into from Sep 27, 2017
Merged

Switch to crowdin #1171

merged 2 commits into from Sep 27, 2017

Conversation

noirbizarre
Copy link
Contributor

This PRs start to document the translation process using crowdin and remove all transifex references

Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

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

A few typo


!!! warning
We only translate strings from the `master` branch of the repository, do not push languages on any other branch because merging translations is incredibly painful.
We only translate strings from the `master` and maintenance branches of the repository,
Do not push languages on any other branch because merging translations is incredibly painful.
Copy link
Contributor

Choose a reason for hiding this comment

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

either no capital D or a point instead of a comma above.



## Existing languages

The command `inv i18n` retrieve all these strings locally that you can then push to Transifex with the command `tx push -s`.
The command `inv i18n` retrieve all these strings locally.
Pushing the changes on any branch will automatically update translatables strings on crowdin..
Copy link
Contributor

Choose a reason for hiding this comment

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

*Crowdin + double point

Copy link
Contributor

Choose a reason for hiding this comment

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

*translatable



## Proposing a new language

Propose the new language on [Transifex][transifex-udata], once accepted you can:
To propose a new language you need to submit a pull-request:
Copy link
Contributor

Choose a reason for hiding this comment

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

No -?

- import the initial translations from transifex: `tx pull -r <language_code>`
- add the corresponding flag in the default theme static assets (use one from [famfamfam flags][famfamfam-flags])
* check that it compiles with the command `inv i18nc` and that it is displayed as you expect on your local instance
- add the corresponding flag in the default theme static assets (use one from [famfamfam flags][famfamfam-flags]) instance
Copy link
Contributor

Choose a reason for hiding this comment

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

for instance?

* submit the pull request

Once it has been accepted, we will also create the new language translation crowdin.
Copy link
Contributor

Choose a reason for hiding this comment

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

in Crowdin?

@noirbizarre noirbizarre merged commit 6ed16a2 into opendatateam:master Sep 27, 2017
@noirbizarre noirbizarre deleted the switch-to-crowdin branch September 27, 2017 15:37
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