Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

fix(settings): change elk to moz social in notifications settings [MOSOWEB-89] #84

Merged
merged 3 commits into from
Nov 15, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(settings): german translations
jpezninjo committed Nov 14, 2023
commit a865702d694cff34822d4448ff596efd5c705dad
6 changes: 3 additions & 3 deletions locales/de-DE.json
Original file line number Diff line number Diff line change
@@ -479,10 +479,10 @@
"unsupported": "Ihr Browser unterstützt keine Push-Benachrichtigungen.",
"warning": {
"enable_close": "Schließen",
"enable_description": "Um Benachrichtigungen zu erhalten, wenn Elk nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen generieren, indem du die Schaltfläche \"@:settings.notifications.show_btn{'\"'} oben aktivierest, sobald sie aktiviert ist.",
"enable_description_desktop": "Um Benachrichtigungen zu erhalten, wenn Elk nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst unter „Einstellungen > Benachrichtigungen > Einstellungen für Push-Benachrichtigungen“ genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen generieren, sobald diese aktiviert sind.",
"enable_description": "Um Benachrichtigungen zu erhalten, wenn Mozilla.social nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen generieren, indem du die Schaltfläche \"@:settings.notifications.show_btn{'\"'} oben aktivierest, sobald sie aktiviert ist.",
Copy link

@cadeyrn cadeyrn Nov 15, 2023

Choose a reason for hiding this comment

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

It was already wrong in the previous version, but since you're already on it: "aktivierest" should be "aktivierst". I am not sure if that's the best word, and it's also not what the English original says. Additionally, "aktivierst" would be repeated twice in quick succession. This doesn't sound good and even makes the sentence a bit confusing.

Suggestion for enable_description: "Um Benachrichtigungen zu erhalten, wenn Mozilla.social nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nÜber die Schaltfläche "@:settings.notifications.show_btn{'"'} oben kannst genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen generieren, sobald diese aktiviert ist."

This would be closer to the English translation and less confusing.

Copy link
Author

Choose a reason for hiding this comment

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

Hi @cadeyrn, thanks for the suggestion! I've put your changes onto a new branch and will ask our product if these fixes can come in. PR is here - #87

"enable_description_desktop": "Um Benachrichtigungen zu erhalten, wenn Mozilla.social nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst unter „Einstellungen > Benachrichtigungen > Einstellungen für Push-Benachrichtigungen“ genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen generieren, sobald diese aktiviert sind.",
"enable_description_mobile": "Du erreichst die Einstellungen auch über das Navigationsmenü „Einstellungen > Benachrichtigungen > Push-Benachrichtigungseinstellungen“.",
"enable_description_settings": "Um Benachrichtigungen zu erhalten, wenn Elk nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen auf demselben Bildschirm generieren, sobald du sie aktivierst.",
"enable_description_settings": "Um Benachrichtigungen zu erhalten, wenn Mozilla.social nicht geöffnet ist, aktiviere Push-Benachrichtigungen. \nDu kannst genau steuern, welche Arten von Interaktionen Push-Benachrichtigungen auf demselben Bildschirm generieren, sobald du sie aktivierst.",
"enable_desktop": "Aktiviere Push-Benachrichtigungen",
"enable_title": "Verpasse nie wieder Benachrichtigungen",
"re_auth": "Offenbar unterstützt dein Server keine Push-Benachrichtigungen. \nVersuche dich abzumelden und erneut anzumelden. Wenn diese Meldung weiterhin angezeigt wird, wende dich an dein Serveradministrator."