-
Notifications
You must be signed in to change notification settings - Fork 149
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
EZP-26834 : document translation workflow with Crowdin #163
Conversation
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.
Just some typos and punctuation
doc/i18n/translation_workflow.md
Outdated
|
||
eZ Platform is translated in several languages with the help of [Crowdin][crowdin-ezplatform]. | ||
|
||
To organize contribution and allow users to install only what they need, we setup a workflow |
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.
To organize contribution and allow users to install only what they need, we set up a workflow
doc/i18n/translation_workflow.md
Outdated
To organize contribution and allow users to install only what they need, we setup a workflow | ||
based on three steps. | ||
|
||
## eZ Platform i18n : Main translation repository |
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.
eZ Platform i18n: Main translation repository
doc/i18n/translation_workflow.md
Outdated
|
||
## eZ Platform i18n : Main translation repository | ||
|
||
To be able to provide a single translation project on our translators platform, we gathered every translation |
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.
To be able to provide a single translation project on our translator platform, we gathered all translation
doc/i18n/translation_workflow.md
Outdated
## eZ Platform i18n : Main translation repository | ||
|
||
To be able to provide a single translation project on our translators platform, we gathered every translation | ||
files is a single repository which is synchronized with Crowdin. |
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.
files in a single repository which is synchronized with Crowdin.
doc/i18n/translation_workflow.md
Outdated
To be able to provide a single translation project on our translators platform, we gathered every translation | ||
files is a single repository which is synchronized with Crowdin. | ||
|
||
Translators can contribute on Crowdin and every contribution are merged in this repository. |
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.
Translators can contribute on Crowdin and every contribution is merged into this repository.
doc/i18n/translation_workflow.md
Outdated
|
||
To allow user to install only what they need, we have splitted every languages in a dedicated package. | ||
|
||
Every translation packages are published on [ezplatform-i18n organisation on github][ezplatform-i18n-org] |
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.
All translation packages are published on [ezplatform-i18n organisation on github][ezplatform-i18n-org]
doc/i18n/translation_workflow.md
Outdated
|
||
Every translation packages are published on [ezplatform-i18n organisation on github][ezplatform-i18n-org] | ||
|
||
**Important** : this packages are read only, they must be updated with the [eZ Platform i18n git split command][ezplatform-i18n]. |
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.
Important: these packages are read only, they must be updated with the [eZ Platform i18n git split command][ezplatform-i18n].
doc/i18n/translation_workflow.md
Outdated
|
||
If you want to install a new language in your project, you just have to install the corresponding package. | ||
|
||
For example if you want to translate your application in Portuguese (pt_PT : the only package supported by our QA team ;)), |
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.
For example, if you want to translate your application into Portuguese (pt_PT: the only package supported by our QA team ;)),
doc/i18n/translation_workflow.md
Outdated
If you want to install a new language in your project, you just have to install the corresponding package. | ||
|
||
For example if you want to translate your application in Portuguese (pt_PT : the only package supported by our QA team ;)), | ||
you just have to run : |
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.
you just have to run:
doc/i18n/translation_workflow.md
Outdated
|
||
composer require ezplatform-i18n/ezplatform-i18n-pt_pt | ||
|
||
and them clean the cache. |
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.
and then clean the cache.
doc/i18n/translation_workflow.md
Outdated
|
||
**Important** : this packages are read only, they must be updated with the [eZ Platform i18n git split command][ezplatform-i18n]. | ||
|
||
## Install a new language on your project |
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.
I would move this paragraph in README.md on INSTALL.md as it is for users and not contributors
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.
@yannickroger I think it make sense to put that here because it's the end of the workflow to validate that the UI is translate
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.
So who is the persona for that documentation?
For me, installing a language is a regular admin user. While understanding how we integrate crowdin is more for contributors. That's why I would move this paragraph to a "admin" doc file. But we could add a sentence here saying to install a language as stated in install.md
doc/i18n/translation_workflow.md
Outdated
To be able to provide a single translation project on our translators platform, we gathered every translation | ||
files is a single repository which is synchronized with Crowdin. | ||
|
||
Translators can contribute on Crowdin and every contribution are merged in this repository. |
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.
You should add a link to the repository
3b823f8
to
a9c2a2d
Compare
@DominikaK done thanks a lot, I guess it's obvious we really need doc writers ;-) @yannickroger I added the link |
doc/i18n/translation_workflow.md
Outdated
|
||
If you want to install a new language in your project, you just have to install the corresponding package. | ||
|
||
For example, if you want to translate your application into Portuguese (pt_PT: the only package supported by our QA team ;)), |
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.
@pcardiga ;-)
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.
One more proofing pass
doc/i18n/translation_workflow.md
Outdated
@@ -4,75 +4,33 @@ | |||
|
|||
eZ Platform is translated in several languages with the help of [Crowdin][crowdin-ezplatform]. | |||
|
|||
To organize contribution and allow users to install only what they need, we set up a workflow | |||
based on three steps. | |||
This documentation presents you the overall concept from the contribution to the distribution with every actor of it. |
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.
This documentation presents the overall concept from contribution to distribution for every participant.
doc/i18n/translation_workflow.md
Outdated
|
||
To be able to provide a single translation project on our translator platform, we gathered all translation | ||
files in a single repository which is synchronized with Crowdin. | ||
This part is made for contributors to eZ Platform. It will explain you how to format your contribution |
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.
This part is for contributors to eZ Platform. It will explain to you how to format your contribution
doc/i18n/translation_workflow.md
Outdated
|
||
### Translation extraction | ||
Then you need to add this new strings in the translation process so they can be translated. |
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.
Then you need to add these new strings to the translation process so they can be translated.
doc/i18n/translation_workflow.md
Outdated
|
||
### Translation extraction | ||
Then you need to add this new strings in the translation process so they can be translated. | ||
This part is made for eZ System team to present the process to send new string for translation. |
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.
This part is made for the eZ Systems team to present the process of sending the new string for translation.
doc/i18n/translation_workflow.md
Outdated
|
||
**Important**: you'll notice that all files will be updated at least to change the date attribute. If this is the | ||
only change to a file, please don't commit it. | ||
When translators have worked, it's time to integrate their translations and distribute it. |
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.
When translators are done, it's time to integrate their translations and distribute them.
|
||
## Introduction | ||
|
||
This documentation is made for eZ Platform integrators which realize projects based on eZ Platform and will present you |
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.
This documentation is made for eZ Platform integrators who realize projects based on eZ Platform. It will present
## Introduction | ||
|
||
This documentation is made for eZ Platform integrators which realize projects based on eZ Platform and will present you | ||
how to install a new package of translation on your project. |
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.
how to install a new package of translation in your project.
|
||
## Introduction | ||
|
||
This documentation is made for eZ System team to present the process to send new string for translation. |
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.
This documentation is made for the eZ Systems team to present the process of sending the new string for translation.
|
||
Before extracting new strings you have to make sure that your eZ Systems bundles are up to date on your eZ Platorm installation. | ||
|
||
Then your project should be in dev mode with developement dependancies installed and ezsystems/ezplatform-i18n up to date. |
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.
Then your project should be in dev mode with development dependencies installed and ezsystems/ezplatform-i18n up to date.
|
||
## Add new strings for translation | ||
|
||
At this point, this only thing you have to do to add the new string in Crowdin is to merge the PR and make sure that |
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.
At this point, the only thing you have to do to add the new string in Crowdin is to merge the PR and make sure that
@DominikaK I'm sorry, but would you mind opening up a pull-request against this one with those changes ? This is a lot of them :) |
Review changes by @DominikaK merged to this pull-request. |
d37cc3a
to
828811b
Compare
Language fixes by @DominikaK.
828811b
to
abc6cac
Compare
Documentation for Jira : https://jira.ez.no/browse/EZP-26834
Description
This documents the translation workflow with crowdin.
Test
Test manually