-
Notifications
You must be signed in to change notification settings - Fork 581
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
chore(i18n): revise translation & i18n keys #1925
Conversation
Umm... leave some more time to check. |
I have updated the main branch to fix the OOM issue. Please rebase on the latest main branch to fix the ci issue |
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.
Please add #[serde(alias = "xx")
to make compatiable with current version, or write a migration to migrate current config.
7382143
to
385f85b
Compare
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.
LGTM
@greenhat616 Thank you for reviewing. But I will keep the draft for a while to check if there are other mistakes in the translation. |
@greenhat616 ready. |
With my best efforts, sincerity, and double-checking for grammar and text style, there may be errors.
And there may be some breaking changes involved, requiring careful review.
cc @keiko233 @greenhat616 @4o3F