Skip to content

[Validator] Update Serbian translation file #30352

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 23, 2019
Merged

[Validator] Update Serbian translation file #30352

merged 1 commit into from
Feb 23, 2019

Conversation

burnmaniac
Copy link
Contributor

@burnmaniac burnmaniac commented Feb 23, 2019

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

I am not sure about some translations, if someone could check these, I would appreciate it.

Also, I've found that singular/plural translations have 3 translations. I am not sure if that's a mistake or not. I removed the third translation, but if I was wrong, I'll be happy to put it back.

@@ -68,39 +68,39 @@
</trans-unit>
<trans-unit id="17">
<source>The mime type of the file is invalid ({{ type }}). Allowed mime types are {{ types }}.</source>
<target>Mime tip datoteke nije validan ({{ type }}). Dozvoljeni mime tipovi su {{ types }}.</target>
<target>MIME tip datoteke nije validan ({{ type }}). Dozvoljeni MIME tipovi su {{ types }}.</target>

Choose a reason for hiding this comment

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

not sure if this should be caps

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also not sure. I've found it in some books on Serbian it's written in full caps, which is why I changed it.

</trans-unit>
<trans-unit id="23">
<source>This value should not be null.</source>
<target>Vrednost ne treba da bude null.</target>
<target>Vrednost ne bi trebalo da bude prazna.</target>

Choose a reason for hiding this comment

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

Ah, hate that null and blank translates to the same word...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Me too, but I couldn't find proper word for it.

</trans-unit>
<trans-unit id="79">
<source>The host could not be resolved.</source>
<target>Nije moguće odrediti poslužitelja.</target>

Choose a reason for hiding this comment

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

not sure if anyone can understand this translation but it is valid...

</trans-unit>
<trans-unit id="81">
<source>This is not a valid Business Identifier Code (BIC).</source>
<target>Ovo nije validan BIC.</target>
Copy link
Member

Choose a reason for hiding this comment

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

don't we want to add something that'd mean "Business Identifier Code"? BIC is clear enough? (same below)

Choose a reason for hiding this comment

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

Don't know if we have a word for BIC, maybe we can literally translate it

Copy link
Contributor Author

@burnmaniac burnmaniac Feb 23, 2019

Choose a reason for hiding this comment

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

Correct, we don't have a word for it. I could translate it as Kodni identifikator banke, but it just sounds wrong. I think that BIC is enough.

@fabpot
Copy link
Member

fabpot commented Feb 23, 2019

Thank you @burnmaniac.

@fabpot fabpot merged commit 9e9a57a into symfony:3.4 Feb 23, 2019
fabpot added a commit that referenced this pull request Feb 23, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

[Validator] Update Serbian translation file

| Q             | A
| ------------- | ---
| Branch?       | 3.4 <!-- see below -->
| Bug fix?      | no
| New feature?  | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #30189  <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | / <!-- required for new features -->

<!--
Write a short README entry for your feature/bugfix here (replace this comment block.)
This will help people understand your PR and can be used as a start of the Doc PR.
Additionally:
 - Bug fixes must be submitted against the lowest branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->
I am not sure about some translations, if someone could check these, I would appreciate it.

Also, I've found that singular/plural translations have 3 translations. I am not sure if that's a mistake or not. I removed the third translation, but if I was wrong, I'll be happy to put it back.

Commits
-------

9e9a57a [Validator] Update Serbian translation file
@burnmaniac burnmaniac deleted the patch-1 branch February 23, 2019 17:22
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