Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(i18n): add comprehensive i18n#31

Merged
nexmoe merged 2 commits intomainfrom
n-branch-1
Nov 16, 2025
Merged

feat(i18n): add comprehensive i18n#31
nexmoe merged 2 commits intomainfrom
n-branch-1

Conversation

@nexmoe
Copy link
Owner

@nexmoe nexmoe commented Nov 15, 2025

No description provided.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 232 to 287
"foundVideos": "在播放列表中找到 {{count}} 个视频",
"groupActive": "{{count}} 个活跃",
"groupErrors": "{{count}} 失败",
"groupSummary": "{{已完成}} / {{总计}}已完成",

Choose a reason for hiding this comment

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

P1 Badge Preserve i18n placeholders while translating

Several of the new locale files translate the variable names inside {{…}}. For example, this entry now uses {{已完成}} and {{总计}} instead of {{completed}} and {{total}}. i18next interpolates only the original keys provided by the code, so these strings will render the raw placeholder text and never show the actual counts when the affected locale is active. Keep the placeholder identifiers identical to the English source and translate only the surrounding words.

Useful? React with 👍 / 👎.

@nexmoe nexmoe merged commit 452132c into main Nov 16, 2025
3 checks passed
@nexmoe nexmoe deleted the n-branch-1 branch November 16, 2025 04:30
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.

1 participant