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

Add i18n and Chinese language #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

alanpeng
Copy link

Add i18n and Chinese language

@wladimiiir
Copy link
Contributor

wladimiiir commented Mar 16, 2025

Thank you for your interest and contribution.

I can see several issues that need to be addressed to get this merged into the main:

  1. to have the proper translation foundation in place, we need to use react-i18next library and utilize useTranslation hook.
  2. translation files need to have proper key organized by their context - for example t("Min Time Between Tool Calls (ms)") needd to be t("settings.mcpSettings.minTimeBetweenToolCalls")
  3. please replace Chinese comments with English ones
  4. there are some other smaller issues that I would comment later, when you decide to work on this

But seeing that multi-language support is something requested, I might work on this next week to prepare the foundation and then I would just kindly ask you to prepare zh.json translation file from the prepared en.json.

Let me know if you would like to work on this further, or you would rather have me take over it. Thanks.

@wladimiiir
Copy link
Contributor

@alanpeng
If you are interested have a look at support-other-languages branch. The foundation is there, what needs to be done is to replace all the strings with t(...) and add the translations in the en.ts, zh.ts files.
Thanks.

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