Skip to content

[Validator] Added Swedish translations #11561

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
Aug 4, 2014
Merged

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Aug 4, 2014

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets part of #11489
License MIT

Added some missing Swedish translations.

Should I make an additional PR to master for the trans-unit id=78 (ref)? Or should I add that in this PR?

@Nyholm
Copy link
Member Author

Nyholm commented Aug 4, 2014

The common typos error is a false positive.

@fabpot
Copy link
Member

fabpot commented Aug 4, 2014

Thank you @Nyholm.

@fabpot fabpot merged commit a361f10 into symfony:2.4 Aug 4, 2014
fabpot added a commit that referenced this pull request Aug 4, 2014
This PR was merged into the 2.4 branch.

Discussion
----------

[Validator] Added Swedish translations

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | part of #11489
| License       | MIT

Added some missing Swedish translations.

Should I make an additional PR to `master` for the `trans-unit id=78` ([ref](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Validator/Resources/translations/validators.en.xlf#L302))? Or should I add that in this PR?

Commits
-------

a361f10 [Validator] Added Swedish translations
@Nyholm Nyholm deleted the swedish_trans branch August 4, 2014 20:11
@Nyholm
Copy link
Member Author

Nyholm commented Aug 4, 2014

Thank you for merging.

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

Successfully merging this pull request may close these issues.

2 participants