Skip to content

fix(i18n): correct malformed ICU plural syntax in Indonesian translation#885

Merged
ascorbic merged 2 commits intoemdash-cms:mainfrom
ahliweb:fix/id-plural-icu-syntax
May 1, 2026
Merged

fix(i18n): correct malformed ICU plural syntax in Indonesian translation#885
ascorbic merged 2 commits intoemdash-cms:mainfrom
ahliweb:fix/id-plural-icu-syntax

Conversation

@ahliweb
Copy link
Copy Markdown
Contributor

@ahliweb ahliweb commented May 1, 2026

What does this PR do?

Fixes a malformed ICU MessageFormat plural in the Indonesian (id) locale file — packages/admin/src/locales/id/messages.po:146. The broken msgstr used {#} (which renders the numeric count) instead of the literal # symbol that the msgid expects, and was missing the one plural form.

Closes #886

Type of change

  • Bug fix
  • Feature (requires maintainer-approved Discussion)
  • Refactor (no behavior change)
  • Translation
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • I have read CONTRIBUTING.md
  • pnpm typecheck passes — N/A (.po file only)
  • pnpm lint passes
  • pnpm test passes — N/A (translation string fix, no behavior change)
  • pnpm format has been run
  • I have added/updated tests for my changes (if applicable)
  • User-visible strings in the admin UI are wrapped for translation — N/A (already translated; this is a fix to an existing translation)
  • I have added a changeset
  • New features link to an approved Discussion: N/A

AI-generated code disclosure

  • This PR includes AI-generated code — model/tool:

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 1, 2026

🦋 Changeset detected

Latest commit: cf8ab74

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages
Name Type
@emdash-cms/admin Patch
emdash Patch
@emdash-cms/cloudflare Patch
@emdash-cms/fixture-perf-site Patch
@emdash-cms/perf-demo-site Patch
@emdash-cms/cache-demo-site Patch
@emdash-cms/auth Patch
@emdash-cms/blocks Patch
@emdash-cms/gutenberg-to-portable-text Patch
@emdash-cms/x402 Patch
create-emdash Patch
@emdash-cms/auth-atproto Patch
@emdash-cms/plugin-embeds Patch

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
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
packages/admin/src/locales/id/messages.po Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 1, 2026

Open in StackBlitz

@emdash-cms/admin

npm i https://pkg.pr.new/@emdash-cms/admin@885

@emdash-cms/auth

npm i https://pkg.pr.new/@emdash-cms/auth@885

@emdash-cms/blocks

npm i https://pkg.pr.new/@emdash-cms/blocks@885

@emdash-cms/cloudflare

npm i https://pkg.pr.new/@emdash-cms/cloudflare@885

emdash

npm i https://pkg.pr.new/emdash@885

create-emdash

npm i https://pkg.pr.new/create-emdash@885

@emdash-cms/gutenberg-to-portable-text

npm i https://pkg.pr.new/@emdash-cms/gutenberg-to-portable-text@885

@emdash-cms/x402

npm i https://pkg.pr.new/@emdash-cms/x402@885

@emdash-cms/plugin-ai-moderation

npm i https://pkg.pr.new/@emdash-cms/plugin-ai-moderation@885

@emdash-cms/plugin-atproto

npm i https://pkg.pr.new/@emdash-cms/plugin-atproto@885

@emdash-cms/plugin-audit-log

npm i https://pkg.pr.new/@emdash-cms/plugin-audit-log@885

@emdash-cms/plugin-color

npm i https://pkg.pr.new/@emdash-cms/plugin-color@885

@emdash-cms/plugin-embeds

npm i https://pkg.pr.new/@emdash-cms/plugin-embeds@885

@emdash-cms/plugin-forms

npm i https://pkg.pr.new/@emdash-cms/plugin-forms@885

@emdash-cms/plugin-webhook-notifier

npm i https://pkg.pr.new/@emdash-cms/plugin-webhook-notifier@885

commit: cf8ab74

@ahliweb
Copy link
Copy Markdown
Contributor Author

ahliweb commented May 1, 2026

I have read the CLA Document and I hereby sign the CLA (gh@ahliweb.com)

1 similar comment
@ahliweb
Copy link
Copy Markdown
Contributor Author

ahliweb commented May 1, 2026

I have read the CLA Document and I hereby sign the CLA (gh@ahliweb.com)

ahliweb added 2 commits May 1, 2026 19:31
The msgstr at packages/admin/src/locales/id/messages.po:146 had invalid
ICU syntax — {#} rendered the numeric count instead of the literal #
symbol, and the one form was missing. Fixed to match the msgid
structure with proper #{1}/#{2} placeholders and both one/other forms.
@ahliweb ahliweb force-pushed the fix/id-plural-icu-syntax branch from 86dc37a to cf8ab74 Compare May 1, 2026 12:31
@ahliweb
Copy link
Copy Markdown
Contributor Author

ahliweb commented May 1, 2026

recheck

Copy link
Copy Markdown
Collaborator

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ascorbic ascorbic merged commit 25128b2 into emdash-cms:main May 1, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Malformed ICU plural syntax in Indonesian (id) locale file

2 participants