i18n(uk): add Ukrainian translations for the admin UI#2120
Conversation
Add uk/messages.po catalog with full admin UI translations and enable the Ukrainian locale in the admin language switcher.
🦋 Changeset detectedLatest commit: 2186c24 The changes in this PR will be included in the next version bump. This PR includes changesets to release 16 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
All contributors have signed the CLA ✍️ ✅ |
Scope checkThis PR changes 8,576 lines across 3 files. Large PRs are harder to review and more likely to be closed without review. If this scope is intentional, no action needed. A maintainer will review it. If not, please consider splitting this into smaller PRs. See CONTRIBUTING.md for contribution guidelines. |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
This is the right kind of change for EmDash: a translation PR that adds a complete uk catalog and enables the locale. The approach fits the existing i18n workflow (locale definition → PO catalog → enabled: true), Ukrainian is correctly treated as LTR, and the 4-form Ukrainian plural rule in the PO header is accurate. The catalog appears complete (1,884 entries with no empty msgstr).
I found one convention issue that should be fixed before merge: the new locale entry is out of alphabetical order in locales.ts. The file documents that entries should be "Source locale first, then alphabetical by English name," but uk (Ukrainian) was inserted between Polish (pl) and Portuguese-Brazil (pt-BR) instead of after Turkish (tr).
I also noted that the new uk/messages.po duplicates two pre-existing corrupted Lingui placeholder-description comments that embed the full source of MenuList.tsx and ContentTypeEditor.tsx. These are already present in the other locale catalogs, so this isn't a regression introduced by the PR, but it does add ~27 KB of source text to the published admin package. That's best cleaned up at the extractor/source JSX level (or as a follow-up), not by hand-editing the catalog.
Once the locale list ordering is corrected, this should be good to merge.
What does this PR do?
Add uk/messages.po catalog with full admin UI translations and enable the Ukrainian locale in the admin language switcher.
Type of change
Checklist
pnpm typecheckpassespnpm lintpassespnpm testpasses (or targeted tests for my change)pnpm formathas been runAI-generated code disclosure
Screenshots / test output