Skip to content

Update manual project order notice wording and translations#5253

Merged
AmethystLiang merged 1 commit into
mainfrom
quick-wording-change
Jun 12, 2026
Merged

Update manual project order notice wording and translations#5253
AmethystLiang merged 1 commit into
mainfrom
quick-wording-change

Conversation

@AmethystLiang

Copy link
Copy Markdown
Contributor

Description

  • Corrects the manual project order default notice from "Manual project order is now default" to "Manual project order is now the default".
  • Integrates complete translations for both ProjectOrderManualDefaultNotice and FolderWorkspaceComposerDialog in Spanish, Japanese, Korean, and Chinese.

Correct a minor grammatical issue in the English notice by adding
"the". Additionally, localize FolderWorkspaceComposerDialog and
ProjectOrderManualDefaultNotice in Spanish, Japanese, Korean, and
Chinese to replace the English fallback strings.
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 38329ca0-2d5c-4686-ba33-dfb7124242db

📥 Commits

Reviewing files that changed from the base of the PR and between 4278fdc and e3e165c.

📒 Files selected for processing (6)
  • src/renderer/src/components/sidebar/ProjectOrderManualDefaultNotice.tsx
  • src/renderer/src/i18n/locales/en.json
  • src/renderer/src/i18n/locales/es.json
  • src/renderer/src/i18n/locales/ja.json
  • src/renderer/src/i18n/locales/ko.json
  • src/renderer/src/i18n/locales/zh.json

📝 Walkthrough

Walkthrough

This pull request updates localization strings across the application. The English source string for ProjectOrderManualDefaultNotice is updated to add the definite article "the", changing from "Manual project order is now default" to "Manual project order is now the default". This change is applied in both the React component and the English locale file. Corresponding updates are made to Spanish, Japanese, Korean, and Chinese locale files for both the ProjectOrderManualDefaultNotice feature and the FolderWorkspaceComposerDialog component.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is incomplete. It lacks the required sections from the repository's description template including Screenshots, Testing checklist, AI Review Report with cross-platform compatibility confirmation, Security Audit, and Notes sections. Add all required template sections: Screenshots (mark as 'No visual change' if applicable), Testing with checkbox completion, AI Review Report confirming cross-platform checks, Security Audit summary, and Notes section.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title accurately and concisely summarizes the main change: updating wording for the manual project order notice and adding translations.
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.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

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 and usage tips.

@AmethystLiang AmethystLiang merged commit 030a35a into main Jun 12, 2026
2 checks passed
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