-
-
Notifications
You must be signed in to change notification settings - Fork 204
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
i18n/de: Add German translations #590
Conversation
Codecov Report
@@ Coverage Diff @@
## master #590 +/- ##
==========================================
+ Coverage 84.13% 84.15% +0.02%
==========================================
Files 474 474
Lines 12518 12518
Branches 1221 1221
==========================================
+ Hits 10532 10535 +3
+ Misses 1893 1890 -3
Partials 93 93 |
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 only noticed two little mistakes. The rest looks fine to me.
src/app/locales/de/settings.yml
Outdated
channels: | ||
find: Kanal finden | ||
none: Keine Kanäle gefunden. | ||
empty: Keine benutzerdefinierten Kanäleinstellungen gesetzt. |
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.
Should probably say "Keine benutzerdefinierten Kanaleinstellungen gesetzt."
src/app/locales/de/settings.yml
Outdated
grouping: | ||
title: Gruppieren | ||
description: Mehrere Benachrichtigungen zusammenfassen. | ||
checkbox: Nur eine Benachrichtigungen auf einmal anzeigen |
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.
Should say "Nur eine Benachrichtigung auf einmal anzeigen"
thanks @skulblakka for taking the time and finding these stupid mistakes 👍 ❤️ |
Opening a PR instead of just pushing the changes to the master branch, so that it can be reviewed...
If nobody will review or even see this within the next days, I will just merge it (probably after this weekend), because I want to finally publish a new release. I want to have this included, even if the translations are not good enough, I don't really care.
Had a bit of trouble finding good translations here and there while not trying to make it sound weird or trying to use more anglicisms than needed. Some texts are also no direct translations, because translating didn't make sense or because I changed it after already translating it.
This also made me realize how inconsistent all of the English texts are and that they need to be improved. Will do this after switching to the new Twitch API, which will require lots of changes anyway.