Skip to content
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

Fixes for Turkish translation files. #314

Closed
wants to merge 15 commits into from

Conversation

krmgns
Copy link
Contributor

@krmgns krmgns commented Dec 19, 2017

No description provided.

@krmgns krmgns changed the title Turkish translation file fixes. Fixes for Turkish translation files. Dec 19, 2017
@@ -17,7 +17,7 @@ msgstr "Entity-Attribute-Value (Varlık-Özellik-Değer)"

#: ../../More/EAV/README.rst:4
msgid "The Entity–attribute–value (EAV) pattern in order to implement EAV model with PHP."
msgstr ""
msgstr "-"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why "-"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

'Cos it does not add anything to translation. It sounds EAV is EAV.

Copy link
Contributor

Choose a reason for hiding this comment

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

@k-gun first of all. If you put "-" in msgstr then this whole sentence "The Entity–attribute–value (EAV) pattern in order to implement EAV model with PHP." will be translated into "-" - this is not cool. Other thing is that this msgid is not a shortcut only but full sentence that should be translated as well.

@@ -50,7 +50,7 @@ msgstr ""

#: ../../More/Repository/README.rst:20
msgid "Laravel Framework"
msgstr ""
msgstr "Aptal bir framework."
Copy link
Contributor

Choose a reason for hiding this comment

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

why translate Laravel Framework? It's a name.

Choose a reason for hiding this comment

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

What! Translation says, 'It is a stupid framework.' ??

@domnikl
Copy link
Contributor

domnikl commented Jun 14, 2018

Closing this as there was no more action on this and it seems that this adds little value and breaks translations as @piotrgradzinski mentioned.

@domnikl domnikl closed this Jun 14, 2018
@elamblin
Copy link

elamblin commented Jul 31, 2018 via email

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.

None yet

5 participants