Skip to content

i18n: complete all non-English locale catalogs - #663

Merged
matteius merged 1 commit into
mainfrom
codex/lightnvr-i18n-656
Oct 18, 2026
Merged

matteius merged 1 commit into
mainfrom
codex/lightnvr-i18n-656

Conversation

@matteius

Copy link
Copy Markdown
Contributor

Closes #656.

What changed

  • Recovered the partial live.* and setupWizard.* translations for seven locales from the stalled DigitalOcean Harness workspace, then completed all missing keys in the 17 non-English catalogs. Each locale now has all 2,645 English keys.
  • Repaired existing translations that had broken RTSP/API example URLs, translated keyboard shortcuts or <stream_name>, omitted API help details, or contained placeholder artifacts.
  • Added a Jest coverage report that warns on future missing keys and fails on orphan keys, invalid interpolation tokens, markup, example URLs, technical names, shortcuts, blank values, or placeholder artifacts.

Validation

  • cd web && npm test -- --runInBand
  • cd web && npx vite build
  • cd web && npx eslint tests/locale-coverage.spec.js
  • Browser spot-check of Settings, Event Routing, and Investigation in German, Japanese, and Arabic using a temporary local LightNVR backend.
  • Confirmed catalog JSON parses and original key order is preserved.

Review note

These translations were machine-produced. Native-speaker review is especially welcome for Japanese, both Chinese variants, Arabic, and Korean. The app still falls back to English for any future missing keys; rollback is a revert.

@codecov

codecov Bot commented Oct 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@matteius
matteius merged commit 4f533c1 into main Oct 18, 2026
13 checks passed
@matteius
matteius deleted the codex/lightnvr-i18n-656 branch October 18, 2026 22:23
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.

[Agent] i18n: fill missing translations in all locales (~44% coverage today)

1 participant