Skip to content

fix(i18n): add 6 missing Japanese translation keys#864

Open
octo-patch wants to merge 2 commits into
ValueCell-ai:mainfrom
octo-patch:fix/ja-i18n-missing-translations
Open

fix(i18n): add 6 missing Japanese translation keys#864
octo-patch wants to merge 2 commits into
ValueCell-ai:mainfrom
octo-patch:fix/ja-i18n-missing-translations

Conversation

@octo-patch
Copy link
Copy Markdown
Contributor

Problem

The Japanese (ja) locale files were missing 6 translation keys that exist in English, Chinese (zh), and Russian (ru). These missing keys would cause the app to fall back to the English strings when running in Japanese.

Missing from src/i18n/locales/ja/cron.json:

  • dialog.channelUnsupportedTag — badge shown when a channel doesn't support scheduled delivery
  • dialog.deliveryChannelUnsupported — message shown in the delivery channel selector
  • toast.deliveryChannelUnsupported — toast notification when an unsupported channel is selected

Missing from src/i18n/locales/ja/settings.json:

  • aiProviders.oauth.getApiKey — "Get API Key" button label in OAuth provider card
  • advanced.telemetry — label for the anonymous usage data toggle
  • advanced.telemetryDesc — description for the telemetry toggle

Solution

Added the missing Japanese translations matching the meaning of the existing en/zh/ru translations:

Key Translation
cron:dialog.channelUnsupportedTag 未対応
cron:dialog.deliveryChannelUnsupported {{channel}} は現在、スケジュールされた送信配信に対応していません。
cron:toast.deliveryChannelUnsupported {{channel}} はまだスケジュール配信に対応していません
settings:aiProviders.oauth.getApiKey APIキーを取得
settings:advanced.telemetry 匿名使用データ
settings:advanced.telemetryDesc ClawXの改善のため、匿名の基本的な使用データの提供を許可する

Testing

  • Both modified JSON files are valid JSON (verified with python3 -m json.tool)
  • TypeScript check passes (pnpm exec tsc -p tsconfig.json --noEmit)
  • Verified all keys now match the English locale

octo-patch added 2 commits April 14, 2026 16:39
Add 6 missing Japanese (ja) translation keys that exist in en, zh, and ru
but were absent from the Japanese locale files:

cron.json:
- dialog.channelUnsupportedTag
- dialog.deliveryChannelUnsupported
- toast.deliveryChannelUnsupported

settings.json:
- aiProviders.oauth.getApiKey
- advanced.telemetry
- advanced.telemetryDesc
Copy link
Copy Markdown
Contributor

@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.

Reviewed commit: 6fa0d628e7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +68 to +72
"slug": "mmx-cli",
"repo": "MiniMax-AI/cli",
"repoPath": "skill",
"ref": "main",
"version": "main",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Badge Remove unintended preinstalled skill from i18n-only change

This patch is scoped to Japanese locale keys, but it also adds a new mmx-cli entry to the preinstalled skills manifest, which changes product behavior by bundling and deploying another third-party skill for users. Because preinstalled skills are installed on startup and can be auto-enabled (ensurePreinstalledSkillsInstalled in electron/utils/skill-config.ts), this introduces an unrelated runtime change in a localization fix and should be split out (or reverted) to avoid shipping unexpected functionality.

Useful? React with 👍 / 👎.

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