Skip to content

feat(i18n): translate identity & permissions to German (CAT-2320)#17749

Open
annadoesdesign wants to merge 1 commit into
masterfrom
feature/cat-2320
Open

feat(i18n): translate identity & permissions to German (CAT-2320)#17749
annadoesdesign wants to merge 1 commit into
masterfrom
feature/cat-2320

Conversation

@annadoesdesign
Copy link
Copy Markdown
Contributor

Summary

Adds the German (DE) translations for the keys added by the identity & permissions extraction (CAT-2234 → PR #17689).

Scope note: the Linear ticket description references a governance.identity namespace, but the actual extraction PR added keys to two existing namespaces — entity.identity and settings.permissions — rather than creating a new file. This PR translates the keys that actually landed.

  • entity.identity.json: +19 keys (group delete / copy URN actions, serviceAccount default-view + delete + roleAssign messages, groups.membersCount plural forms)
  • settings.permissions.json: +6 keys (policy activate / deactivate / delete / remove / save success messages and the delete confirmation dialog)

Translation choices follow conventions already used in these files:

  • Policy → Richtlinie, Group → Gruppe
  • Owner / Service Account / URN kept as-is
  • German quotation marks „…" for inline quoted UI strings
  • i18next _one / _other plural forms with German singular / plural

Closes CAT-2320.

Test plan

  • node scripts/check-translations.mjs --lang de — both namespaces now report complete
  • All {{name}}, {{error}}, {{role}}, {{formattedCount}} interpolations preserved verbatim
  • prettier --write on both files is a no-op (already correctly formatted)
  • Spot-check the Manage Users & Groups and Manage Permissions pages in DE locale (reviewer)

Made with Cursor

The identity & permissions extraction (CAT-2234 / PR #17689) added keys
to two existing namespaces — entity.identity and settings.permissions —
rather than a new governance.identity namespace as the ticket title
implies. This PR fills in the matching DE keys (formal Sie form).

- entity.identity.json: +19 keys (group delete/copy URN actions,
  serviceAccount default-view + delete + roleAssign messages,
  groups.membersCount plural forms)
- settings.permissions.json: +6 keys (policy
  activate/deactivate/delete/remove/save success messages and the
  delete confirmation dialog)

Translation choices follow conventions already used in these files:
- Policy → Richtlinie, Group → Gruppe, Owner kept as-is,
  Service Account kept as-is, URN kept as-is
- German quotation marks „…" for inline quoted UI strings
- i18next _one/_other plural forms preserved with German singular/plural

Verified with scripts/check-translations.mjs --lang de: both namespaces
now report complete. All {{name}}, {{error}}, {{role}}, and
{{formattedCount}} interpolations preserved.

Closes CAT-2320

Co-authored-by: Cursor <cursoragent@cursor.com>
@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Jun 4, 2026

✅ Meticulous spotted 0 visual differences across 1423 screens tested: view results.

Meticulous evaluated ~10 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit edca7a7 feat(i18n): translate identity & permissions to German (CAT-2320). This comment will update as new commits are pushed.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Bundle Report

Changes will increase total bundle size by 1.83kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 23.54MB 1.83kB (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/de-*.js 1.83kB 49.51kB 3.84%

Files in assets/de-*.js:

  • ./src/i18n/locales/de/settings.permissions.json → Total Size: 8.97kB

  • ./src/i18n/locales/de/entity.identity.json → Total Size: 11.06kB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants