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

PlatformUI translation doc #818

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

StephaneDiot
Copy link
Contributor

Description

Update the developer's tasks of the README.md about how to generate translated strings in PlatformUI

@yannickroger
Copy link
Contributor

You could add a note reminding telling people to be careful not to commit translation while if only the date of generation has been updated.

Copy link
Contributor

@yannickroger yannickroger left a comment

Choose a reason for hiding this comment

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

With feedback taken into account

@@ -157,3 +157,12 @@ Then, you can run the unit tests with:
```
$ php vendor/bin/phpunit
```

* Translation

Copy link
Contributor

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 doc explaining the translation process.


* Translation

To generate the .xlf translation file (in PlatformUIBundle/Ressources/translations)
Copy link
Contributor

Choose a reason for hiding this comment

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

If you added new translation strings,

@sonarcloud
Copy link

sonarcloud bot commented Jul 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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

Successfully merging this pull request may close these issues.

2 participants