8000 [Validator] missing translation for Uuid constraint · Issue #27984 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content
8000

[Validator] missing translation for Uuid constraint #27984

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.

Sign up for GitHub

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

Jump to bottom
Closed
garak opened this issue Jul 18, 2018 · 1 comment
Closed

[Validator] missing translation for Uuid constraint #27984

garak opened this issue Jul 18, 2018 · 1 comment
Labels
Bug Status: Needs Review Validator

Comments

@garak
Copy link
Contributor
garak commented Jul 18, 2018

Symfony version(s) affected: 4.1.1 (and likely all previous versions)

Description
Message of Uuid constraint is missing translations in all languages

How to reproduce
Just try to validate an Uuid with locale enabled.
Expected result: a translated error message.
Actual result: "This is not a valid UUID."

Possible Solution
We need to add such messages to all validators.XX.xlf in validator translations (I can provide a PR for Italian language only)

Additional context
None
@fabpot
Copy link
Member
fabpot commented Jul 19, 2018

Please, open a PR for Italian. Other languages will follow.

garak added a commit to garak/symfony that referenced this issue Jul 19, 2018
@garak
add missing uuid validator translation
bf04358 
See issue symfony#27984
This was referenced Jul 19, 2018
Closed
Closed
Merged
fabpot added a commit that referenced this issue Jul 19, 2018
@fabpot
minor #27992 add missing translation for uuid validator (garak)
08a49bc 
This PR was merged into the 2.8 branch.

Discussion
----------

add missing translation for uuid validator

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

Commits
-------

7cf840c add missing translation for uuid validator
@xabbuh xabbuh closed this as completed Jul 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Needs Review Validator
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fabpot @garak @xabbuh @carsonbot
0