Skip to content

New Crowdin updates#986

Merged
steveiliop56 merged 30 commits into
mainfrom
l10n_main
Jul 9, 2026
Merged

New Crowdin updates#986
steveiliop56 merged 30 commits into
mainfrom
l10n_main

Conversation

@steveiliop56

@steveiliop56 steveiliop56 commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Expanded localization coverage for authentication and authorization screens across many languages.
    • Added new translated UI text for Tailscale login/logout flows and “Back to login”.
    • Added “Quick Actions” labels (language/theme/logout and provider selection) and new consent scope labels/descriptions for Phone and Address.
  • Bug Fixes
    • Updated authorization error messaging to reflect invalid or missing request parameters.
    • Refreshed selected existing translated strings for improved wording/consistency.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 9, 2026
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 8b1b7d34-e0e0-4d6b-9886-c026eb8b930d

📥 Commits

Reviewing files that changed from the base of the PR and between 48e9e7b and 44f0c01.

📒 Files selected for processing (4)
  • frontend/src/lib/i18n/locales/nl-NL.json
  • frontend/src/lib/i18n/locales/pl-PL.json
  • frontend/src/lib/i18n/locales/sr-SP.json
  • frontend/src/lib/i18n/locales/zh-CN.json
✅ Files skipped from review due to trivial changes (2)
  • frontend/src/lib/i18n/locales/nl-NL.json
  • frontend/src/lib/i18n/locales/pl-PL.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • frontend/src/lib/i18n/locales/sr-SP.json
  • frontend/src/lib/i18n/locales/zh-CN.json

📝 Walkthrough

Walkthrough

Locale translation entries were updated across many frontend locale JSON files. The authorization error key was renamed to authorizeErrorInvalidParams, and new strings were added for back-to-login, phone/address scopes, Tailscale login/logout messaging, and Quick Actions labels.

Changes

Locale translation updates

Layer / File(s) Summary
Auth copy refresh
frontend/src/lib/i18n/locales/*.json
Renames authorizeErrorMissingParams to authorizeErrorInvalidParams and updates surrounding login, redirect, authorization, and error text in the affected locales.
Back-to-login and scopes
frontend/src/lib/i18n/locales/*.json
Adds backToLoginButton plus phone and address scope name/description entries after the existing OpenID/email/profile/groups scope strings.
Tailscale auth strings
frontend/src/lib/i18n/locales/*.json
Adds Tailscale login/logout translation keys, including device-name prompts, success/failure messages, and logout subtitles.
Quick Actions labels
frontend/src/lib/i18n/locales/*.json
Adds Quick Actions translation keys for language/theme selection, logout, and provider labels for Local, LDAP, and OAuth.

Estimated code review effort: 2 (Simple) | ~12 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is related to the change, but it is too vague to describe the translation-only scope of the PR. Use a more specific title like "Update Crowdin locale translations" or "Refresh i18n locale strings from Crowdin".
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch l10n_main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 6

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@frontend/src/lib/i18n/locales/fr-FR.json`:
- Line 74: The `authorizeErrorInvalidParams` entry in the French locale is still
in English, so restore it to a proper French translation to match the
surrounding keys. Update the value in `fr-FR.json` for
`authorizeErrorInvalidParams` with the intended French message, keeping the key
name unchanged and ensuring it remains consistent with the earlier
`authorizeErrorMissingParams` translation.

In `@frontend/src/lib/i18n/locales/it-IT.json`:
- Line 74: The authorizeErrorInvalidParams entry in the Italian locale is still
in English; update this key in it-IT.json to use the proper Italian translation
and keep it consistent with the surrounding i18n strings. Use the
authorizeErrorInvalidParams symbol to locate the untranslated value and restore
the Italian text replacing the former authorizeErrorMissingParams wording.

In `@frontend/src/lib/i18n/locales/nl-NL.json`:
- Line 89: The Dutch localization string for loginTailscaleDescription contains
a typo where “jeTailscale-verbinding” is missing a space. Update the value in
the nl-NL locale entry to use the correct phrasing “je Tailscale-verbinding”,
keeping the rest of the message unchanged and ensuring the key
loginTailscaleDescription remains the same.

In `@frontend/src/lib/i18n/locales/pl-PL.json`:
- Line 80: Fix the typo in the i18n string for profileScopeDescription by
correcting the transposed letters in the Polish text. Update the localized value
in the pl-PL locale entry so it reads naturally with “profilu”; use the
profileScopeDescription key to locate it.

In `@frontend/src/lib/i18n/locales/sr-SP.json`:
- Line 102: The new quickActionsProviderLocal translation in the sr-SP locale is
using Latin script instead of the Cyrillic script used throughout this file.
Update the value in the sr-SP locale entry for quickActionsProviderLocal to the
Cyrillic equivalent so it matches the rest of the translations and stays
consistent with the locale’s script.

In `@frontend/src/lib/i18n/locales/zh-CN.json`:
- Line 101: The zh-CN locale entry for quickActionsTitle is using a Traditional
Chinese form, so update the value in the zh-CN translation file to Simplified
Chinese. Locate the quickActionsTitle key in the locale JSON and change the text
from the Traditional variant to the Simplified Chinese equivalent so it matches
the rest of the zh-CN translations.
🪄 Autofix (Beta)

✅ Autofix completed


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 62697429-31b0-4c9f-b9be-bf908b1459fd

📥 Commits

Reviewing files that changed from the base of the PR and between 8299362 and 48e9e7b.

📒 Files selected for processing (29)
  • frontend/src/lib/i18n/locales/af-ZA.json
  • frontend/src/lib/i18n/locales/ar-SA.json
  • frontend/src/lib/i18n/locales/ca-ES.json
  • frontend/src/lib/i18n/locales/cs-CZ.json
  • frontend/src/lib/i18n/locales/da-DK.json
  • frontend/src/lib/i18n/locales/de-DE.json
  • frontend/src/lib/i18n/locales/el-GR.json
  • frontend/src/lib/i18n/locales/es-ES.json
  • frontend/src/lib/i18n/locales/fi-FI.json
  • frontend/src/lib/i18n/locales/fr-FR.json
  • frontend/src/lib/i18n/locales/he-IL.json
  • frontend/src/lib/i18n/locales/hu-HU.json
  • frontend/src/lib/i18n/locales/it-IT.json
  • frontend/src/lib/i18n/locales/ja-JP.json
  • frontend/src/lib/i18n/locales/ko-KR.json
  • frontend/src/lib/i18n/locales/nl-NL.json
  • frontend/src/lib/i18n/locales/no-NO.json
  • frontend/src/lib/i18n/locales/pl-PL.json
  • frontend/src/lib/i18n/locales/pt-BR.json
  • frontend/src/lib/i18n/locales/pt-PT.json
  • frontend/src/lib/i18n/locales/ro-RO.json
  • frontend/src/lib/i18n/locales/ru-RU.json
  • frontend/src/lib/i18n/locales/sr-SP.json
  • frontend/src/lib/i18n/locales/sv-SE.json
  • frontend/src/lib/i18n/locales/tr-TR.json
  • frontend/src/lib/i18n/locales/uk-UA.json
  • frontend/src/lib/i18n/locales/vi-VN.json
  • frontend/src/lib/i18n/locales/zh-CN.json
  • frontend/src/lib/i18n/locales/zh-TW.json

Comment thread frontend/src/lib/i18n/locales/fr-FR.json
Comment thread frontend/src/lib/i18n/locales/it-IT.json
Comment thread frontend/src/lib/i18n/locales/nl-NL.json Outdated
Comment thread frontend/src/lib/i18n/locales/pl-PL.json Outdated
Comment thread frontend/src/lib/i18n/locales/sr-SP.json Outdated
Comment thread frontend/src/lib/i18n/locales/zh-CN.json Outdated
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Fixes Applied Successfully

Fixed 4 file(s) based on 4 unresolved review comments.

A stacked PR containing fixes has been created.

  • Stacked PR: #987
  • Files modified:
  • frontend/src/lib/i18n/locales/nl-NL.json
  • frontend/src/lib/i18n/locales/pl-PL.json
  • frontend/src/lib/i18n/locales/sr-SP.json
  • frontend/src/lib/i18n/locales/zh-CN.json

Time taken: 3m 17s

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 9, 2026
@steveiliop56 steveiliop56 merged commit 405d910 into main Jul 9, 2026
5 checks passed
@steveiliop56 steveiliop56 deleted the l10n_main branch July 9, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants