Skip to content

i18n(ru): AI translation update β€” sync with messages.pot (2026-06-17) - #12963

Merged
mekarpeles merged 2 commits into
masterfrom
i18n/ru
Jun 18, 2026
Merged

i18n(ru): AI translation update β€” sync with messages.pot (2026-06-17)#12963
mekarpeles merged 2 commits into
masterfrom
i18n/ru

Conversation

@mekarpeles

Copy link
Copy Markdown
Member

Summary

Syncs the Russian (ru) translation file with the current `messages.pot` template and fills previously untranslated strings.

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, particularly for tone and idiomatic accuracy. Format strings (`%(name)s`, `%(count)d`, etc.) and HTML markup were preserved verbatim.

Changes

  • Fixed 14 pre-existing format-string errors (8 validator errors + fuzzy entries with incompatible placeholders)
  • Fixed 1 non-fuzzy entry missing `%(ip)s` in href attribute
  • Ran `scripts/i18n-messages update ru` to sync with current `messages.pot`
  • Filled 11 previously untranslated msgid entries
  • Russian was already 93.95% complete β€” this brings it to 100% of non-fuzzy strings

Validation

  • Format-string errors fixed: 0 errors βœ“
  • `i18n-messages validate ru` β€” 0 errors βœ“
  • `i18n-messages compile ru` β€” compiled successfully βœ“
  • `pre-commit run --files openlibrary/i18n/ru/messages.po` β€” clean βœ“
  • `test_po_files.py -k ru` β€” 178 passed βœ“

Reviewer guidance

Please spot-check translations for:

  1. Accuracy of meaning (does the translation convey the intent?)
  2. Natural phrasing using Π²Ρ‹ form (formal Russian)
  3. Correct plural forms for the target language
  4. Preserved format strings and HTML

Native speakers: please edit the `.po` file directly on this branch.

πŸ€– Generated by `claude-sonnet-4-6`

Copilot AI review requested due to automatic review settings June 17, 2026 19:21

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 bc6dae0 into master Jun 18, 2026
7 of 8 checks passed
@mekarpeles
mekarpeles deleted the i18n/ru branch June 18, 2026 04:36
mekarpeles added a commit that referenced this pull request Jun 19, 2026
When a patron verifies their account via /account/verify?t=TOKEN:
- On success: redirect to /account/books instead of the home page
- On error + already logged in: redirect to /account/books instead of
  /account/create (handles the "link already used" case gracefully)
- Honor a ?redirect=... query param passed in the verify URL

Fixes #12963
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