Skip to content

New Crowdin updates#993

Closed
steveiliop56 wants to merge 8 commits into
mainfrom
l10n_main
Closed

New Crowdin updates#993
steveiliop56 wants to merge 8 commits into
mainfrom
l10n_main

Conversation

@steveiliop56

@steveiliop56 steveiliop56 commented Jul 11, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Localization
    • Updated translated interface text across Dutch, Polish, Serbian, and Simplified Chinese locales.
    • Refined the Serbian label for local quick-action providers.
    • Updated the Chinese quick-actions heading.
    • Adjusted Polish profile description wording.
    • Standardized the representation of a Dutch login description without changing its displayed text.

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

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Four frontend locale JSON files were updated: one Dutch entry’s representation changed, a Polish translation contains a typo change, and Serbian and Chinese quick-action translations changed.

Changes

Locale translation updates

Layer / File(s) Summary
Translation entry updates
frontend/src/lib/i18n/locales/{nl-NL,pl-PL,sr-SP,zh-CN}.json
Updated four localized entries, including Serbian Latin text, Chinese wording, a Polish typo, and Dutch line representation.

Estimated code review effort: 1 (Trivial) | ~2 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 localization changes, but it is too generic to clearly describe the main updates. Use a more specific title, such as mentioning the updated Crowdin locale translations or localization strings.
✅ 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: 4

🤖 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/nl-NL.json`:
- Line 89: Correct the Dutch loginTailscaleDescription translation by inserting
a space between “je” and “Tailscale”, preserving the rest of the user-facing
text unchanged.

In `@frontend/src/lib/i18n/locales/pl-PL.json`:
- Line 80: Correct the typo in the `profileScopeDescription` translation value
by changing “porfilu” to “profilu”; leave the rest of the authorization message
unchanged.

In `@frontend/src/lib/i18n/locales/sr-SP.json`:
- Line 102: Update the quickActionsProviderLocal translation to use the Cyrillic
value "Локално", matching the script convention used by quickActionsTitle and
the surrounding Serbian locale entries.

In `@frontend/src/lib/i18n/locales/zh-CN.json`:
- Line 101: Update the quickActionsTitle value in the zh-CN locale to the
Simplified Chinese, software-appropriate translation “快速操作”, preserving its use
as the quick-actions accessibility label.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e8aac930-879b-4fa5-934c-65eae090761c

📥 Commits

Reviewing files that changed from the base of the PR and between 009ae30 and 9a430cd.

📒 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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant