Skip to content

i18n(ja): AI translation update — fill remaining 14 strings + fix pre-existing errors (2026-06-18)#12982

Merged
mekarpeles merged 1 commit into
masterfrom
i18n/ja
Jun 22, 2026
Merged

i18n(ja): AI translation update — fill remaining 14 strings + fix pre-existing errors (2026-06-18)#12982
mekarpeles merged 1 commit into
masterfrom
i18n/ja

Conversation

@mekarpeles

@mekarpeles mekarpeles commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Fills in the remaining 14 untranslated Japanese strings and clears 5 pre-existing msgstr entries that had validation errors. The previous Japanese translation PR (#12925) brought ja to ~96%; this PR completes the remaining gaps.

AI attribution: All new translations in this PR were generated by claude-sonnet-4-6. They should be reviewed by a native speaker before merging.

Changes

  • Cleared 5 pre-existing invalid msgstr entries (placeholder incompatibilities from old translations)
  • Filled 14 previously untranslated msgid entries (UI labels, toggle component docs, search strings)
  • 1 string left untranslated (contains a control character \x08 in msgid — not translateable)

Validation

  • i18n-messages validate ja — 0 errors ✓
  • i18n-messages compile ja — compiled successfully ✓
  • pre-commit run --files openlibrary/i18n/ja/messages.po — clean ✓

Reviewer guidance

Please spot-check translations for:

  1. Natural phrasing for UI elements (toggle component, search results strings)
  2. Preserved format strings (%s, %(label)s, etc.)

🤖 Generated by claude-sonnet-4-6

Copilot AI review requested due to automatic review settings June 18, 2026 21:13

Copilot AI left a comment

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.

Copilot wasn't able to review any files in this pull request.

@mekarpeles mekarpeles merged commit 638aa70 into master Jun 22, 2026
7 of 8 checks passed
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