Skip to content

update i18n text of zh#79

Open
shincurry wants to merge 2 commits intoTryGhost:mainfrom
shincurry:i18n-zh
Open

update i18n text of zh#79
shincurry wants to merge 2 commits intoTryGhost:mainfrom
shincurry:i18n-zh

Conversation

@shincurry
Copy link
Copy Markdown

@shincurry shincurry commented Apr 4, 2026

Note

Low Risk
Low risk: updates only locales/zh.json string translations with no logic changes. Main risk is minor UI copy/formatting regressions from incorrect wording or placeholders.

Overview
Fills in previously-empty Chinese (zh) i18n strings across the UI (navigation, membership/subscription flows, pagination, and post/issue labels).

Also tweaks a few existing translations for consistency (e.g., read-time phrasing) and updates the sample email from a localized address to zhangwei@example.com.

Reviewed by Cursor Bugbot for commit 4ddbdff. Bugbot is set up for automated code reviews on this repo. Configure here.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 905dafdd-0483-435e-b0b4-c086a7054baa

📥 Commits

Reviewing files that changed from the base of the PR and between d66a622 and 4ddbdff.

📒 Files selected for processing (1)
  • locales/zh.json
🚧 Files skipped from review as they are similar to previous changes (1)
  • locales/zh.json

Walkthrough

This pull request updates locales/zh.json by replacing many previously empty Chinese translation values with localized text. Changes cover pagination labels, navigation CTAs (including arrow variants), issue/post collection wording with preserved placeholders (e.g., {numberOfIssues}, {numberOfPosts}), authentication and subscription messages (status, errors, CTAs), form prompts, miscellaneous UI labels (menu toggles, “Show more”, RSS/publisher), and an adjusted topic context placeholder ("in {primaryTag}").

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'update i18n text of zh' is vague and generic, using non-descriptive terms that don't clearly convey the meaningful content of the changeset. Consider a more specific title like 'Add Chinese translations for navigation and subscription flows' or 'Localize UI strings in Chinese (zh) locale' to better communicate the primary change.
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed The description is detailed and clearly related to the changeset, explaining the purpose (filling empty Chinese translations), scope (navigation, membership flows, pagination), and noting any tweaks or risks.
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

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.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d66a622. Configure here.

Copy link
Copy Markdown

@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: 2

🧹 Nitpick comments (1)
locales/zh.json (1)

64-64: Refine brand phrase for smoother wording.

Line [64] “由 Ghost 强力驱动” reads awkwardly in Chinese product copy. “由 Ghost 驱动” is more natural.

Suggested copy tweak
-    "Powered by Ghost": "由 Ghost 强力驱动",
+    "Powered by Ghost": "由 Ghost 驱动",
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@locales/zh.json` at line 64, Replace the current Chinese translation value
for the key "Powered by Ghost" in locales/zh.json ("由 Ghost 强力驱动") with the more
natural phrasing "由 Ghost 驱动"; update the string value only and keep the JSON
key intact.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@locales/zh.json`:
- Around line 21-22: The translations for the keys "All episodes" and "All
episodes →" use “集数” while other entries use “剧集”; pick one term and make them
consistent — change the values for "All episodes" and "All episodes →" to use
the same word as the rest of the file (e.g., "所有剧集" and "所有剧集 →") and update any
other zh.json entries that use the alternate term so the UI uses a single
Chinese term for episodes throughout.
- Line 92: Update the Chinese translation value for the i18n key "Subscription
confirmed!" in locales/zh.json to a more natural phrase such as "订阅已确认!" (or
"订阅成功!"); locate the entry with the key "Subscription confirmed!" and replace
the current value "订阅确认!" with the chosen improved copy.

---

Nitpick comments:
In `@locales/zh.json`:
- Line 64: Replace the current Chinese translation value for the key "Powered by
Ghost" in locales/zh.json ("由 Ghost 强力驱动") with the more natural phrasing "由
Ghost 驱动"; update the string value only and keep the JSON key intact.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8f336208-2095-4404-9445-fc2fb07c2292

📥 Commits

Reviewing files that changed from the base of the PR and between d68331d and d66a622.

📒 Files selected for processing (1)
  • locales/zh.json

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