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: add vietnam language #951

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

thinhbg2812-2
Copy link

@thinhbg2812-2 thinhbg2812-2 commented Dec 2, 2024

Add Vietnamese for @valibot/i18n

Here my brother: @hongky-1994

Signed-off-by: ThomasBui <[email protected]>
@thinhbg2812-2 thinhbg2812-2 force-pushed the vn-language branch 3 times, most recently from 90d4985 to 0547fb7 Compare December 2, 2024 10:39
@fabian-hiller
Copy link
Owner

Will @hongky-1994 review your translations?

@fabian-hiller fabian-hiller self-assigned this Dec 4, 2024
@fabian-hiller fabian-hiller added the enhancement New feature or request label Dec 4, 2024
Copy link

@hongky-1994 hongky-1994 left a comment

Choose a reason for hiding this comment

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

I think some changes should be made to improve understanding of error messages.

packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
packages/i18n/src/vi.ts Outdated Show resolved Hide resolved
thinhbg2812-2 and others added 12 commits December 5, 2024 02:37
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
Update translation

Co-authored-by: Ky_pham <[email protected]>
@fabian-hiller
Copy link
Owner

Is this ready to be merged, or should another native speaker do a final review?

@thinhbg2812-2
Copy link
Author

I think it's ready; i asked him pretty much about the naming and context, and he has high credit in team xD

@fabian-hiller
Copy link
Owner

Ok. Thank you! I will review and merge it later this month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants