-
Notifications
You must be signed in to change notification settings - Fork 289
feat(language): 添加越南语语言包 #3378
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: feat_v3.x
Are you sure you want to change the base?
feat(language): 添加越南语语言包 #3378
Conversation
概述新增越南语区域设置文件 变更
代码审查工作量评估🎯 1 (简单) | ⏱️ ~5 分钟 需要特别注意的方面:
诗
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this 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
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/locales/vi-VN.ts(1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/locales/vi-VN.ts (1)
src/locales/base.ts (1)
BaseLang(1-143)
🔇 Additional comments (5)
src/locales/vi-VN.ts (5)
1-3: 文件结构正确。导入语句和类型声明符合预期,确保了类型安全。
101-111: 验证公司特定的引用是否合适。
basePlaceholder: 'Lên JD tìm sản phẩm tốt'包含了 "JD"(京东)的品牌引用。请确认这是否是预期的行为,还是应该使用更通用的占位符文本。如果此 UI 库用于通用场景,建议使用不包含特定公司名称的文本。
33-43: 日历翻译实现正确。周几缩写格式和月份标题函数的实现符合越南语规范。
96-100: 动态消息函数实现正确。
additionalReview和additionalImages函数签名与BaseLang接口要求匹配。
1-144: 语言包整体结构完整。该文件完整实现了
BaseLang接口的所有必需字段,结构符合其他语言包的模式。除了已标记的小问题外,这是一个很好的越南语国际化实现。
| countdown: { | ||
| day: 'Ngày', | ||
| hour: 'Giờ', | ||
| minute: 'Phút', | ||
| second: '', | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
second 字段值为空字符串。
countdown.second 的值是空字符串,这可能是遗漏。建议使用 "Giây"(与 datepicker.seconds 保持一致)。
应用此修改:
countdown: {
day: 'Ngày',
hour: 'Giờ',
minute: 'Phút',
- second: '',
+ second: 'Giây',
},📝 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.
| countdown: { | |
| day: 'Ngày', | |
| hour: 'Giờ', | |
| minute: 'Phút', | |
| second: '', | |
| }, | |
| countdown: { | |
| day: 'Ngày', | |
| hour: 'Giờ', | |
| minute: 'Phút', | |
| second: 'Giây', | |
| }, |
🤖 Prompt for AI Agents
In src/locales/vi-VN.ts around lines 59 to 64, the countdown.second value is an
empty string; replace it with the Vietnamese word "Giây" (to match
datepicker.seconds) so countdown.second: 'Giây' and save the file preserving
existing object formatting.
| addresslist: { | ||
| addAddress: 'Tạo địa chỉ ', | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addAddress 字段值末尾有额外空格。
addAddress: 'Tạo địa chỉ ' 末尾有一个空格,这可能是意外输入。建议删除尾随空格以保持一致性。
应用此修改:
addresslist: {
- addAddress: 'Tạo địa chỉ ',
+ addAddress: 'Tạo địa chỉ',
},📝 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.
| addresslist: { | |
| addAddress: 'Tạo địa chỉ ', | |
| }, | |
| addresslist: { | |
| addAddress: 'Tạo địa chỉ', | |
| }, |
🤖 Prompt for AI Agents
In src/locales/vi-VN.ts around lines 93 to 95, the addAddress string contains an
unintended trailing space ("Tạo địa chỉ "); remove the trailing space so the
value is "Tạo địa chỉ" to keep translations consistent and avoid accidental
whitespace in UI output.
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单
Summary by CodeRabbit