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

Spanish translation #543

Merged
merged 3 commits into from
Jan 12, 2017
Merged

Spanish translation #543

merged 3 commits into from
Jan 12, 2017

Conversation

bediazu
Copy link
Contributor

@bediazu bediazu commented Jan 11, 2017

i have ready a spanish translate

@bediazu bediazu changed the title Spanish Translate Spanish translation Jan 11, 2017
@@ -0,0 +1,4 @@
{
"descriptionEnglish": "Español (Spanish)",
Copy link
Collaborator

Choose a reason for hiding this comment

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

The description here should be totally in english.

e.g.: "descriptionEnglish": "Spanish (from Spain) or just Spanish is fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ohh ok! Sorry, no problem! :D

#: Sources/Client/Client_NetHandler.cpp:127
msgctxt "Client"
msgid "{0} captured {1}'s intel"
msgstr "{0} capturó la intel {1}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I only know very basic spanish, but shouldn't the translation be "{0} capturó la intel de {1}"?

Then again, there is a high chance that I might be wrong, because I don't know much spanish.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Basically is the same, anyway I will change it because they understood. Thank you :D 👍

A simply change in "{0} captured {1}'s intel"
@feikname feikname added this to the 0.1.1 milestone Jan 12, 2017
@feikname
Copy link
Collaborator

Seems like everything is fine now, thanks for the contribution! 👍

Your translation will probably be incorporated in the next version ;)

@yvt yvt merged commit d59c605 into yvt:master Jan 12, 2017
@bediazu bediazu deleted the brunodiazu-translate branch January 12, 2017 03:07
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.

3 participants