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

fix: show Vietnamese in its own language #1454

Merged
merged 1 commit into from
May 13, 2023

Conversation

yanCode
Copy link
Contributor

@yanCode yanCode commented May 12, 2023

vm

Changes in this PR:

  1. In the dropdown list, it should show the language name in its own language. I mean, it should show Tiếng Việt, rather than Vietnamese in its English translation.

  2. Apart from 1, I found lots of formatted code,in app/locales/cs.ts and app/locales/ru.ts resulting many lines of changes in my PR, I did the research, I found these changes in this PR is caused by lint-staged, I mean for the reason I don't know, these files were committed without running the pre-commit in husky hook , I tested locally the whole pipeline of lint-staged works fine in my local env. maybe these 2 files were directly committed via the webpage of git hub? we'd better make sure no commits can bypass husky hook for git.

@yanCode
Copy link
Contributor Author

yanCode commented May 13, 2023

Did a quick test, as you can see here: yanCode@8c7f441 I can make a commit to the codebase via github web UI, which doesn't trigger husky at all. which can likely introduce code format issues as in this PR.

@yanCode yanCode changed the title fix: show Vietnamese in it's own language fix: show Vietnamese in its own language May 13, 2023
@Yidadaa Yidadaa merged commit 6bb0166 into ChatGPTNextWeb:main May 13, 2023
1 check passed
@yanCode yanCode deleted the fix/i18n branch May 13, 2023 06:54
Endlessworld pushed a commit to Endlessworld/ChatGPT-Next-Web that referenced this pull request May 17, 2023
fix: show Vietnamese in its own language
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.

None yet

2 participants