i18n(nl): translate built-in widget UI strings#2114
Open
mikepage wants to merge 2 commits into
Open
Conversation
PR emdash-cms#2057 added new source strings for the built-in widget UI (Recent Posts, category list, tag cloud, search form, archives) plus their prop-field and option labels. The merge-time locale extraction added these msgids to the catalogs untranslated. Fill in the Dutch translations for all 18 new widget msgids in the nl catalog, following the existing register (informal, sentence case, infinitive labels). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: 5be6803 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 |
github-actions
Bot
added
area/admin
size/S
review/needs-review
Contributor
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
@emdash-cms/admin
@emdash-cms/auth
@emdash-cms/auth-atproto
@emdash-cms/blocks
@emdash-cms/cloudflare
@emdash-cms/contentful-to-portable-text
emdash
create-emdash
@emdash-cms/gutenberg-to-portable-text
@emdash-cms/plugin-cli
@emdash-cms/plugin-types
@emdash-cms/registry-client
@emdash-cms/registry-lexicons
@emdash-cms/registry-verification
@emdash-cms/sandbox-workerd
@emdash-cms/x402
@emdash-cms/plugin-ai-moderation
@emdash-cms/plugin-atproto
@emdash-cms/plugin-audit-log
@emdash-cms/plugin-color
@emdash-cms/plugin-embeds
@emdash-cms/plugin-field-kit
@emdash-cms/plugin-forms
@emdash-cms/plugin-webhook-notifier
commit: |
![emdashbot[bot]](https://avatars.githubusercontent.com/in/3255304?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
The approach is exactly right: this is a scoped translation PR that fills the 18 untranslated Dutch msgstrs introduced by #2057, without touching any source code. As a follow-up translation PR, updating only the nl catalog is the correct process.
I checked:
- The diff is limited to
packages/admin/src/locales/nl/messages.po. - All 18 new widget msgids from
packages/admin/src/components/Widgets.tsxare translated (Display a list of recent posts,Display category list,Display tag cloud,Search form,Monthly/yearly archives,Number of posts,Show thumbnails,Show date,Show post count,Show hierarchy,Show count,Maximum tags,Placeholder text,Group by,Limit,Monthly,Yearly, andRecent Posts). - The Dutch register matches the existing catalog: informal tone, sentence case, and the established
{noun} tonenpattern forShow …labels (e.g., existingShow token→Token tonen). - Shared labels such as
Categories,Tags,Search, andArchiveswere already translated and are not duplicated or conflicting. - No empty
msgstrentries remain forWidgets.tsx-sourced strings.
No bugs, regressions, or AGENTS.md violations found. LGTM.
github-actions
Bot
added
review/needs-rereview
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Follow-up to #2035 review comment.
PR #2057 (i18n(admin): localize built-in widget UI labels) introduced new
msgsource strings for the built-in widget UI (Recent Posts, category list, tag cloud, search form, monthly/yearly archives) plus their prop-field and option labels, but shipped no translations. The merge-timepnpm locale:extractadded these 18 msgids to the catalogs untranslated.This PR fills in the Dutch translations for all 18 new widget msgids in
packages/admin/src/locales/nl/messages.po, following the existing catalog's register: informal, sentence case, infinitive button/label forms.Closes #
Type of change
Checklist
pnpm typecheckpasses — n/a, translation-only (.po); no code changedpnpm lintpasses — n/a,.pofile not linted; no new diagnosticspnpm testpasses — n/a, translation-onlypnpm formathas been run — n/a,.pofiles are not oxfmt-managedmessages.pochanges except in translation PRs — a workflow extracts catalogs on merge tomain. — This is a translation PR; only thenlcatalog changed.@emdash-cms/adminpatchAI-generated code disclosure
Screenshots / test output
Translation-only change to
packages/admin/src/locales/nl/messages.po(18 msgstr filled in). No behavior or code changed.