Skip to content

Updated Vietnamese translations#28760

Merged
9larsons merged 14 commits into
TryGhost:mainfrom
honhatduy:main
Jun 24, 2026
Merged

Updated Vietnamese translations#28760
9larsons merged 14 commits into
TryGhost:mainfrom
honhatduy:main

Conversation

@honhatduy

Copy link
Copy Markdown
Contributor
  • Updated the Vietnamese translations to include recently added strings:

The Stripe product title used for tips and donations
When no plans are available
Updates and announcements email preference

  • Refined some existing translations for better clarity and conciseness.

@coderabbitai

coderabbitai Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

Vietnamese locale strings are updated across three files in ghost/i18n/locales/vi/. In comments.json, three existing translations are reworded: "Read more replies", "See full discussion", and "View in admin". In ghost.json, the previously empty value for "Support {siteTitle}" is filled. In portal.json, four previously empty values are filled: the unsubscribe confirmation message, "Occasional updates from {siteTitle}", "Sorry, no paid plans are available.", and "Updates & announcements".

Possibly related PRs

  • TryGhost/Ghost#27839: Modifies the same i18n translation catalogs including comments.json and portal-related locale strings.
  • TryGhost/Ghost#28185: Directly edits the same Vietnamese locale keys in ghost/i18n/locales/vi/comments.json, including "Read more replies" and "See full discussion".

Suggested labels

affects:i18n

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Updated Vietnamese translations' directly summarizes the main change in the changeset, which updates Vietnamese translation strings across multiple locale files.
Description check ✅ Passed The description is related to the changeset, mentioning updates to Vietnamese translations including recently added strings and refinements to existing 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 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.

@github-actions github-actions 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.

🌐 Automated translation review

Verdict:Looks good — no concerns flagged

Reviewed 8 translations across 3 files.

The translations are clean, accurate, and consistent with the existing Vietnamese locale. The three new strings ("Occasional updates from {siteTitle}", "Sorry, no paid plans are available.", and "Updates & announcements") are all translated naturally and correctly, placeholders are intact, and the refined strings in comments.json and ghost.json read well. Cảm ơn bạn!


Advisory review by i18n-review-bot. Non-blocking — a maintainer still owns the merge decision, and the bot cannot approve PRs on its own. Translator expertise wins where there is doubt.

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

🤖 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 `@ghost/i18n/locales/vi/comments.json`:
- Line 83: In the comments.json Vietnamese locale file, the translation for the
"View in admin" key currently uses the English loanword "dashboard" in the
phrase "Xem trong dashboard". Replace this with the proper Vietnamese equivalent
"bảng quản trị" to maintain consistency with how similar administrative and
management terms are translated elsewhere in the Vietnamese locale files (such
as "Quản lý" for "Manage"). Update the translation value to "Xem trong bảng quản
trị" to ensure proper localization.
🪄 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

Run ID: 67c52a40-7451-4b12-a699-2f28c6303fbc

📥 Commits

Reviewing files that changed from the base of the PR and between 74e1e3e and dad6b24.

📒 Files selected for processing (3)
  • ghost/i18n/locales/vi/comments.json
  • ghost/i18n/locales/vi/ghost.json
  • ghost/i18n/locales/vi/portal.json

Comment thread ghost/i18n/locales/vi/comments.json
@9larsons 9larsons enabled auto-merge (squash) June 24, 2026 13:54
@nx-cloud

nx-cloud Bot commented Jun 24, 2026

Copy link
Copy Markdown

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit 251615e

Command Status Duration Result
nx run @tryghost/admin-x-settings:test:acceptance ✅ Succeeded 10m 10s View ↗
nx run ghost:test:ci:integration ✅ Succeeded 2m 35s View ↗
nx build @tryghost/announcement-bar ✅ Succeeded <1s View ↗
nx build @tryghost/activitypub ✅ Succeeded 2s View ↗
nx build @tryghost/signup-form ✅ Succeeded <1s View ↗
nx build @tryghost/comments-ui ✅ Succeeded <1s View ↗
nx build @tryghost/portal ✅ Succeeded 1s View ↗
nx build @tryghost/sodo-search ✅ Succeeded <1s View ↗
Additional runs (12) ✅ Succeeded ... View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-06-24 14:06:28 UTC

@9larsons 9larsons merged commit 4667c91 into TryGhost:main Jun 24, 2026
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants