Skip to content

New Crowdin updates#1907

Merged
elibosley merged 25 commits intomainfrom
l10n_main
Mar 13, 2026
Merged

New Crowdin updates#1907
elibosley merged 25 commits intomainfrom
l10n_main

Conversation

@elibosley
Copy link
Member

@elibosley elibosley commented Mar 13, 2026

Summary by CodeRabbit

  • Documentation
    • Added new localization strings for the onboarding exit modal across all supported languages, ensuring users receive relevant guidance in their preferred language.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 13, 2026

Walkthrough

Added a new localization key onboarding.modal.exit.internalBootDescription across 21 locale files (ar, bn, ca, cs, da, de, es, fr, hi, hr, hu, it, ja, ko, lv, nl, no, pl, pt, ro, ru, sv, uk, zh) with translated descriptions for internal boot behavior during onboarding exit. No existing keys modified or logic changes introduced.

Changes

Cohort / File(s) Summary
Localization: Internal Boot Description
web/src/locales/ar.json, web/src/locales/bn.json, web/src/locales/ca.json, web/src/locales/cs.json, web/src/locales/da.json, web/src/locales/de.json, web/src/locales/es.json, web/src/locales/fr.json, web/src/locales/hi.json, web/src/locales/hr.json, web/src/locales/hu.json, web/src/locales/it.json, web/src/locales/ja.json, web/src/locales/ko.json, web/src/locales/lv.json, web/src/locales/nl.json, web/src/locales/no.json, web/src/locales/pl.json, web/src/locales/pt.json, web/src/locales/ro.json, web/src/locales/ru.json, web/src/locales/sv.json, web/src/locales/uk.json, web/src/locales/zh.json
Added new translation key onboarding.modal.exit.internalBootDescription with descriptive text explaining internal boot configuration and manual reboot instructions. Applied consistently across all supported language locales.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Whiskers twitching with localization glee,
Twenty-one tongues now speak of boots with thee,
From Arabic scripts to Chinese we cheer,
Each language knows: "Please reboot, my dear!"

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'New Crowdin updates' is vague and generic, using non-descriptive terminology that does not convey meaningful information about the specific changeset. Replace with a more specific title that describes the actual change, such as 'Add onboarding.modal.exit.internalBootDescription localization keys' to clarify the purpose of these localization additions.
✅ Passed checks (2 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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch l10n_main
📝 Coding Plan
  • Generate coding plan for human review comments

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

@github-actions
Copy link
Contributor

This plugin has been deployed to Cloudflare R2 and is available for testing.
Download it at this URL:

https://preview.dl.unraid.net/unraid-api/tag/PR1907/dynamix.unraid.net.plg

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@web/src/locales/zh.json`:
- Line 127: The translation string keyed by
"onboarding.modal.exit.internalBootDescription" contains an extra space after
the Chinese full stop (。) before "请"; update the value to remove that space so
the sentence reads "...不会切换到该启动设备。请在方便时手动重新启动以完成应用此更改。" preserving the rest of
the string and punctuation.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: bb7941ae-a612-4a2b-a09f-dcfa6821782b

📥 Commits

Reviewing files that changed from the base of the PR and between 23f7836 and 03b31c3.

📒 Files selected for processing (24)
  • web/src/locales/ar.json
  • web/src/locales/bn.json
  • web/src/locales/ca.json
  • web/src/locales/cs.json
  • web/src/locales/da.json
  • web/src/locales/de.json
  • web/src/locales/es.json
  • web/src/locales/fr.json
  • web/src/locales/hi.json
  • web/src/locales/hr.json
  • web/src/locales/hu.json
  • web/src/locales/it.json
  • web/src/locales/ja.json
  • web/src/locales/ko.json
  • web/src/locales/lv.json
  • web/src/locales/nl.json
  • web/src/locales/no.json
  • web/src/locales/pl.json
  • web/src/locales/pt.json
  • web/src/locales/ro.json
  • web/src/locales/ru.json
  • web/src/locales/sv.json
  • web/src/locales/uk.json
  • web/src/locales/zh.json

"onboarding.modal.closeAriaLabel": "关闭入门",
"onboarding.modal.exit.title": "退出入门?",
"onboarding.modal.exit.description": "您可以现在跳过设置,稍后从仪表板继续。",
"onboarding.modal.exit.internalBootDescription": "内部启动已配置完成。您现在将在所选的启动驱动器上看到一个数据分区,但 Unraid 在您重新启动且同时连接当前的 USB 启动设备和所选的内部启动驱动器之前不会切换到该启动设备。 请在方便时手动重新启动以完成应用此更改。",
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove the extra space after Chinese punctuation in this string.
There’s an unnecessary space after , which creates inconsistent Chinese typography in UI copy.

✂️ Proposed fix
-  "onboarding.modal.exit.internalBootDescription": "内部启动已配置完成。您现在将在所选的启动驱动器上看到一个数据分区,但 Unraid 在您重新启动且同时连接当前的 USB 启动设备和所选的内部启动驱动器之前不会切换到该启动设备。 请在方便时手动重新启动以完成应用此更改。",
+  "onboarding.modal.exit.internalBootDescription": "内部启动已配置完成。您现在将在所选的启动驱动器上看到一个数据分区,但 Unraid 在您重新启动且同时连接当前的 USB 启动设备和所选的内部启动驱动器之前不会切换到该启动设备。请在方便时手动重新启动以完成应用此更改。",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"onboarding.modal.exit.internalBootDescription": "内部启动已配置完成。您现在将在所选的启动驱动器上看到一个数据分区,但 Unraid 在您重新启动且同时连接当前的 USB 启动设备和所选的内部启动驱动器之前不会切换到该启动设备。 请在方便时手动重新启动以完成应用此更改。",
"onboarding.modal.exit.internalBootDescription": "内部启动已配置完成。您现在将在所选的启动驱动器上看到一个数据分区,但 Unraid 在您重新启动且同时连接当前的 USB 启动设备和所选的内部启动驱动器之前不会切换到该启动设备。请在方便时手动重新启动以完成应用此更改。",
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@web/src/locales/zh.json` at line 127, The translation string keyed by
"onboarding.modal.exit.internalBootDescription" contains an extra space after
the Chinese full stop (。) before "请"; update the value to remove that space so
the sentence reads "...不会切换到该启动设备。请在方便时手动重新启动以完成应用此更改。" preserving the rest of
the string and punctuation.

@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.81%. Comparing base (23f7836) to head (03b31c3).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1907   +/-   ##
=======================================
  Coverage   50.81%   50.81%           
=======================================
  Files        1023     1023           
  Lines       70545    70545           
  Branches     7652     7652           
=======================================
  Hits        35848    35848           
  Misses      34573    34573           
  Partials      124      124           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@elibosley elibosley merged commit 2f3f34b into main Mar 13, 2026
13 checks passed
@elibosley elibosley deleted the l10n_main branch March 13, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant