Skip to content

feat(i18n): add vietnamese localization#195

Merged
flick9000 merged 3 commits into
mainfrom
feat-viLanguage
Jun 5, 2026
Merged

feat(i18n): add vietnamese localization#195
flick9000 merged 3 commits into
mainfrom
feat-viLanguage

Conversation

@flick9000
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds Vietnamese language support (vi) to both the app and the website, including Astro configuration updates, a new flag icon, and translation files. The reviewer identified a minor inconsistency in both vi.json files where trailing periods were included in the titles for iexplorer, faxscan, and mediaplayer, and suggested removing them to maintain consistency with other titles.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread app/src/i18n/locales/vi.json Outdated
Comment on lines +60 to +62
"debloat.features.iexplorer": "Tắt Internet Explorer.",
"debloat.features.faxscan": "Tắt Fax và Scan.",
"debloat.features.mediaplayer": "Tắt Media Player.",
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.

medium

The title keys for iexplorer, faxscan, and mediaplayer have trailing periods, which is inconsistent with all other title keys in the localization file (which do not end with a period). Please remove the trailing periods to maintain consistency.

Suggested change
"debloat.features.iexplorer": "Tắt Internet Explorer.",
"debloat.features.faxscan": "Tắt Fax và Scan.",
"debloat.features.mediaplayer": "Tắt Media Player.",
"debloat.features.iexplorer": "Tắt Internet Explorer",
"debloat.features.faxscan": "Tắt Fax và Scan",
"debloat.features.mediaplayer": "Tắt Media Player",

Comment thread website/src/i18n/locales/vi.json Outdated
Comment on lines +60 to +62
"debloat.features.iexplorer": "Tắt Internet Explorer.",
"debloat.features.faxscan": "Tắt Fax và Scan.",
"debloat.features.mediaplayer": "Tắt Media Player.",
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.

medium

The title keys for iexplorer, faxscan, and mediaplayer have trailing periods, which is inconsistent with all other title keys in the localization file (which do not end with a period). Please remove the trailing periods to maintain consistency.

Suggested change
"debloat.features.iexplorer": "Tắt Internet Explorer.",
"debloat.features.faxscan": "Tắt Fax và Scan.",
"debloat.features.mediaplayer": "Tắt Media Player.",
"debloat.features.iexplorer": "Tắt Internet Explorer",
"debloat.features.faxscan": "Tắt Fax và Scan",
"debloat.features.mediaplayer": "Tắt Media Player",

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying winscript with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6d3d26f
Status:⚡️  Build in progress...

View logs

@flick9000 flick9000 merged commit cc8adc0 into main Jun 5, 2026
1 check was pending
@flick9000 flick9000 deleted the feat-viLanguage branch June 5, 2026 15:16
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.

2 participants