Skip to content

fix(i18n): add missing Turkish error translations - #12494

Open
MustafaKemal0146 wants to merge 2 commits into
zitadel:mainfrom
MustafaKemal0146:fix/missing-error-translations
Open

fix(i18n): add missing Turkish error translations#12494
MustafaKemal0146 wants to merge 2 commits into
zitadel:mainfrom
MustafaKemal0146:fix/missing-error-translations

Conversation

@MustafaKemal0146

@MustafaKemal0146 MustafaKemal0146 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Which Problems Are Solved

  • The Turkish common translations were missing four error-message keys available in the English reference file.
  • Affected errors could fall back to English or expose an untranslated localization key.

How the Problems Are Solved

  • Adds natural Turkish translations for:
    • Errors.PermissionDenied
    • Errors.User.MFA.RecoveryCodes.Empty
    • Errors.Session.IDMissing
    • Errors.Session.NotFound
  • Compares all common locale files against en.yaml.
  • Leaves other languages unchanged because they require native-speaker review.

Additional Changes

None.

Additional Context

  • All 22 YAML locale files parse successfully after the change.
  • The Turkish locale has no missing Errors.* keys compared with en.yaml.
  • git diff --check passes.
  • Go tests were not run because the Go toolchain is unavailable in the local environment.
  • ro.yaml has a pre-existing hierarchy/indentation problem and was intentionally left unchanged to keep this change minimal.
  • Closes Add missing translations for error messages #11841

@vercel

vercel Bot commented Jul 26, 2026

Copy link
Copy Markdown

@MustafaKemal0146 is attempting to deploy a commit to the zitadel Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Jul 26, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 28, 2026 8:22am UTC

Request Review

@hifabienne hifabienne added the os-contribution This is a contribution from our open-source community label Jul 26, 2026
@nx-cloud

nx-cloud Bot commented Jul 26, 2026 •
edited
Loading

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit fe5389a

Command Status Duration Result
nx run @zitadel/docs:build ✅ Succeeded 3m 17s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗

☁️ Nx Cloud last updated this comment at 2026-09-28 08:19:10 UTC

@MustafaKemal0146

MustafaKemal0146 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

Updated with latest main. Turkish translations are aligned and docs build is passing.

@MustafaKemal0146
MustafaKemal0146 marked this pull request as ready for review September 28, 2026 09:06
Copilot AI lite review requested due to automatic review settings September 28, 2026 09:06
Copilot AI reviewed Sep 28, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Adds four missing Turkish error translations, aligning tr.yaml with the English reference.

Changes:

  • Adds permission-denied and MFA recovery-code translations.
  • Adds missing session ID and not-found translations.
File Description
internal/​static/​i18n/​tr.yaml Adds the four Turkish error translations.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch was successfully deployed

1 active deployment
Preview – docs — fe5389a6 Deployed Sep 28, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

os-contribution This is a contribution from our open-source community

Projects

Product Management
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add missing translations for error messages

3 participants

@MustafaKemal0146 @hifabienne