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 translations #7235

Merged
merged 8 commits into from
Dec 26, 2019
Merged

Spanish translations #7235

merged 8 commits into from
Dec 26, 2019

Conversation

Moonstroke82
Copy link
Contributor

@Moonstroke82 Moonstroke82 commented Oct 10, 2019

When merged this pull request will:
Add Spanish translation for
Dogtags
UI
AI
Vehicles
Weapons
Zeus

Turkish translation for
UI (incomplete)
AI (incomplete)

Added some more translations into spanish for ai,ui,vehicles, weapons and zeus
Turksih Translation for AI Aircraft Main Map and Medical Feedback started
@TheMagnetar
Copy link
Member

TheMagnetar commented Oct 10, 2019

Can you please properly fill out the template PR ?

@Moonstroke82
Copy link
Contributor Author

Added description

@commy2
Copy link
Contributor

commy2 commented Oct 10, 2019

How come you speak Turkish and Spanish?

@commy2 commy2 added this to the 3.13.0 milestone Oct 10, 2019
@commy2 commy2 added the kind/translation Release Notes: **TRANSLATIONS:** label Oct 10, 2019
@commy2 commy2 changed the title Spanish Dogtags added Spanish + Turkish translations Oct 10, 2019
@Moonstroke82
Copy link
Contributor Author

Haha my buddy and best man is Turkish and my wife used to life in Spain :)

addons/ai/stringtable.xml Outdated Show resolved Hide resolved
addons/common/stringtable.xml Outdated Show resolved Hide resolved
addons/dogtags/stringtable.xml Outdated Show resolved Hide resolved
addons/ui/stringtable.xml Outdated Show resolved Hide resolved
addons/ui/stringtable.xml Outdated Show resolved Hide resolved
addons/zeus/stringtable.xml Outdated Show resolved Hide resolved
addons/zeus/stringtable.xml Outdated Show resolved Hide resolved
addons/zeus/stringtable.xml Outdated Show resolved Hide resolved
addons/zeus/stringtable.xml Outdated Show resolved Hide resolved
addons/common/stringtable.xml Outdated Show resolved Hide resolved
addons/ai/stringtable.xml Outdated Show resolved Hide resolved
Copy link
Contributor

@654wak654 654wak654 left a comment

Choose a reason for hiding this comment

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

I don't like the Turkish translations. They're good word by word translations but some of them don't convey the same meaning as their English counterpart.

For example the word "On" is translated as "üzerinde" but in this context it should've been "Açık" (Also why lower case?).

Overall it feels like some really low effort translations, and since there aren't many of them I think we could do without them.

@PabstMirror PabstMirror modified the milestones: 3.13.0, 3.13.1 Dec 17, 2019
PabstMirror and others added 4 commits December 19, 2019 11:12
@PabstMirror PabstMirror modified the milestones: 3.13.1, 3.13.0 Dec 19, 2019
@PabstMirror PabstMirror changed the title Spanish + Turkish translations Spanish translations Dec 26, 2019
@PabstMirror PabstMirror dismissed stale reviews from 654wak654 and TheMagnetar December 26, 2019 02:41

dropped turk translations

@PabstMirror PabstMirror merged commit 8e24d95 into acemod:master Dec 26, 2019
@PabstMirror PabstMirror modified the milestones: 3.13.0, 3.13.0-temp1 Dec 30, 2019
@PabstMirror PabstMirror added this to the 3.13.0 milestone Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/translation Release Notes: **TRANSLATIONS:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants