10000 [Validator] Add the missing translations for the French (fr) locale by mdelanoue · Pull Request #30199 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Validator] Add the missing translations for the French (fr) locale #30199

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

Merged
merged 1 commit into from
Feb 12, 2019
Merged

[Validator] Add the missing translations for the French (fr) locale #30199

merged 1 commit into from
Feb 12, 2019

Conversation

mdelanoue
Copy link
@mdelanoue mdelanoue commented Feb 12, 2019
Q A
Branch? 3.4
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #30164
License MIT
Doc PR -

To simplify things, translated files contains all the keys of the English file in master branch. That's why our translations in 3.4 branch contain the keys id = 84 and id = 85 even if that is not used in 3.4 branch.

I propose to also include that unused key in the French file to make it easier to keep things in sync.

@javiereguiluz
Copy link
Member

@DelanoueMartin yes, we're updating the English file in the 3.4 branch as well to add the missing translations (#30157)

@fabpot
Copy link
Member
fabpot commented Feb 12, 2019

Thank you @DelanoueMartin.

@fabpot fabpot merged commit 9a86504 into symfony:3.4 Feb 12, 2019
fabpot added a commit that referenced this pull request Feb 12, 2019
…(fr) locale (Martin)

This PR was merged into the 3.4 branch.

Discussion
----------

[Validator] Add the missing translations for the French (fr) locale

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #30164
| License       | MIT
| Doc PR        | -

To simplify things, translated files contains all the keys of the French file in `master` branch. That's why our translations in 3.4 branch contain the keys `id = 84` and `id = 85` even if that is not used in 3.4 branch.

I propose to also include that unused key in the French file to make it easier to keep things in sync.

Commits
-------

9a86504 [Validator] Add the missing translations for the French (fr) locale
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.

5 participants
0