Skip to content
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

feat(locale): add Viet language #2986

Merged
merged 3 commits into from
Dec 26, 2024
Merged

feat(locale): add Viet language #2986

merged 3 commits into from
Dec 26, 2024

Conversation

HigherOrderLogic
Copy link
Contributor

πŸ”— Linked issue

N/A

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

Add the new Viet locale.

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

Sorry, something went wrong.

@HigherOrderLogic HigherOrderLogic changed the title feat(locale): aff Viet language feat(locale): add Viet language Dec 26, 2024
Copy link

pkg-pr-new bot commented Dec 26, 2024 β€’

npm i https://pkg.pr.new/@nuxt/ui@2986

commit: 2dbf787

@@ -17,6 +17,7 @@ function getEmojiFlag(locale: string): string {
ja: 'jp',
ko: 'kr',
uk: 'ua',
vn: 'vi',
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be?

Suggested change
vn: 'vi',
vi: 'vn',

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, vi is the language code, vn is the country.

Verified

This commit was signed with the committer’s verified signature.
aduh95 Antoine du Hamel
@benjamincanac benjamincanac merged commit ffba108 into nuxt:v3 Dec 26, 2024
2 checks passed
@benjamincanac
Copy link
Member

Thanks! 😊

@benjamincanac benjamincanac added the v3 #1289 label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3 #1289
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants