chore(fe): update German translations#3864
Conversation
|
@marc0olo this PR may already be merged by the time you see it, but if you spot any translation mistakes feel free to leave comments or suggestions here — they'll be picked up by AI in a future run. Besides specific fixes, broader feedback is also welcome (e.g., tone, terminology preferences, style guidelines) — these will be reviewed and applied across all future translations. |
There was a problem hiding this comment.
Pull request overview
Updates the German Lingui catalog to cover recently added/changed SSO-related UI messages so German users see localized error strings instead of falling back to the default message.
Changes:
- Added German translation for an SSO timeout error message.
- Replaced the old “couldn’t reach domain” SSO message with the newer “couldn’t load SSO settings…” message + German translation.
- Removed a couple of now-obsolete SSO-related msgids from
de.po.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| msgid "Couldn't reach {domainInput}. Check the spelling and your network." | ||
| msgstr "{domainInput} konnte nicht erreicht werden. Überprüfen Sie die Schreibweise und Ihre Netzwerkverbindung." | ||
| msgid "Couldn't load SSO settings from {domainInput}. Ask your SSO admin to check that /.well-known/ii-openid-configuration is reachable." | ||
| msgstr "SSO-Einstellungen von {domainInput} konnten nicht geladen werden. Bitten Sie Ihren SSO-Administrator zu prüfen, ob /.well-known/ii-openid-configuration erreichbar ist." |
There was a problem hiding this comment.
Fixed — added the missing comma before the infinitive clause. The translation now reads: "Bitten Sie Ihren SSO-Administrator, zu prüfen, ob /.well-known/ii-openid-configuration erreichbar ist."
| msgid "Couldn't load SSO settings from {domainInput}. Ask your SSO admin to check that /.well-known/ii-openid-configuration is reachable." | ||
| msgstr "SSO-Einstellungen von {domainInput} konnten nicht geladen werden. Bitten Sie Ihren SSO-Administrator zu prüfen, ob /.well-known/ii-openid-configuration erreichbar ist." | ||
|
|
There was a problem hiding this comment.
Acknowledged. The
@dfinity/internet-identity@0.1.0 extract
lingui extract --clean
Catalog statistics for src/frontend/src/lib/locales/{locale}:
┌─────────────┬─────────────┬─────────┐
│ Language │ Total count │ Missing │
├─────────────┼─────────────┼─────────┤
│ �[1men�[22m (source) │ 428 │ - │
│ de │ 428 │ 0 │
│ es │ 428 │ 2 │
│ fr │ 428 │ 2 │
│ id │ 428 │ 2 │
│ it │ 428 │ 2 │
│ nl │ 428 │ 2 │
│ pl │ 428 │ 2 │
│ ru │ 428 │ 2 │
│ uk │ 428 │ 2 │
│ ur │ 428 │ 2 │
└─────────────┴─────────────┴─────────┘
(Use "�[33mnpm run extract�[39m" to update catalogs with new messages.)
(Use "�[33mnpm run compile�[39m" to compile catalogs for production. Alternatively, use bundler plugins: https://lingui.dev/ref/cli#compiling-catalogs-in-ci) command was run as part of this fix. Per this PR's scope (German translations only), only was staged and committed. The broader catalog sync across all locales (en.po and others) is outside the scope of this translation PR — that would be better addressed in a separate upstream commit.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Thanks for the review overview @copilot-pull-request-reviewer — the inline feedback items (missing comma and catalog sync question) have both been addressed in the latest commit. |
New translations were missing for `de`. This PR adds the missing translations.
Changes