-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
feat: german translations for dynamax #1054
Conversation
packages/locales/lib/human/de.json
Outdated
"dynamax": "Dynamax", | ||
"stations_subtitle": "Kraftquellen auf der Map anzeigen", | ||
"dynamax_subtitle": "Dyna-Raids auf der Map anzeigen", | ||
"override": "Dyna-Raid Override" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this "override" just used for Dyna-Raid or for every other preset-filter like raids?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TurtIeSocks did you publish this key on purpose like that? instead dyna_override
like for raid_override
? :)
packages/locales/lib/human/de.json
Outdated
"stations": "Dynamax", | ||
"stations_options": "Kraftquellen Optionen", | ||
"all_stations": "alle Kraftquellen", | ||
"search_battles": "Suche Dyna-Raids", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not being consistent with search phrases? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's the same in English. Stations/Battles used in backend, translated to Power Spots/Dynamax or Max Battles in clients.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TurtIeSocks was more about the translation itself, he changed the other search_key values, but this matches the old phrasing :)
it's not about dyna/spots or something else :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left it generic in this case. In English it's just Override
without any category attached to it. Dynamax or Max Battles as a prefix was just too long and feels unnecessary with the way the UI is setup.
packages/locales/lib/human/de.json
Outdated
"unknown_station": "unbekannte Kraftquelle", | ||
"exclude_battle": "Dyna-Raids ausschließen", | ||
"station": "Kraftquelle", | ||
"stations": "Dynamax", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Singular = Kraftquelle
plural = Dynamax ?
I think this should be "Kraftquellen"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove these changes from this PR? As they're unrelated to the core goal and I plan to implement this for stations a bit differently.
this? |
packages/locales/lib/human/de.json
Outdated
"stations_filters": "Dynamax Filter Einstellungen", | ||
"stations_options": "Kraftquellen Optionen", | ||
"all_stations": "alle Kraftquellen", | ||
"search_battles": "Suche Dyna-Raids", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"search_battles": "Suche Dyna-Raids", | |
"search_battles": "Dyna-Raids durchsuchen", |
packages/locales/lib/human/de.json
Outdated
"search_raids": "Raids durchsuchen", | ||
"search_eggs": "Eier durchsuchen", | ||
"search_quests": "Quests durchsuchen", | ||
"search_lures": "Lockmodulen durchsuchen", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"search_lures": "Lockmodulen durchsuchen", | |
"search_lures": "Lockmodule durchsuchen", |
One more comment.. but I think you need to rebase on latest and maybe drop zhe commits of the tsx file which have been reverted first... Guess those are causing the conflict |
cdaca12
to
826ef80
Compare
🎉 This PR is included in version 1.35.0-develop.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.