Skip to content

feat(i18n): fill 725 empty Spanish translations in po/es.po [POC - ignore for now]#5147

Closed
Copilot wants to merge 4 commits into
hg/i18n-translation-agent-skillfrom
copilot/update-spanish-translations
Closed

feat(i18n): fill 725 empty Spanish translations in po/es.po [POC - ignore for now]#5147
Copilot wants to merge 4 commits into
hg/i18n-translation-agent-skillfrom
copilot/update-spanish-translations

Conversation

Copilot AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor
  • Identified root cause: npm run gettext-extract + gettext-update-po run from superdesk-client-core standalone regenerated a smaller POT (core-only), and msgattrib --no-obsolete then stripped 1,373 entries from planning/analytics
  • Restored po/superdesk.pot from the develop branch baseline (includes ../superdesk-planning and ../superdesk-analytics refs)
  • Restored po/es.po from develop baseline (4,273 entries, preserving all planning/analytics translations)
  • Re-applied translations only to entries that were empty in the develop baseline (700 entries filled)
  • Verified: 0 existing translations overwritten, 0 entries missing vs develop baseline
  • Validated with grunt nggettext_compile — passes

eos87 and others added 3 commits April 15, 2026 11:42
Add AI-assisted PO translation workflow and tooling: new repository skill (.github/skills/translate-po/SKILL.md) and i18n docs (docs/i18n/*) including agent instructions, Spanish glossary, translation rules, and workflow guidance. Introduce a small Node task (tasks/gettext-update-po.js) that wraps GNU gettext tools (msgmerge + msgattrib) to sync po/<lang>.po from po/superdesk.pot, plus an npm script (gettext-update-po) in package.json. The script validates inputs, checks files, and emits a helpful error if gettext tools are missing.

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.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@eos87 eos87 marked this pull request as ready for review April 16, 2026 08:58
@eos87 eos87 changed the title feat(i18n): fill 725 empty Spanish translations in po/es.po feat(i18n): fill 725 empty Spanish translations in po/es.po [POC - ignore for now] Apr 16, 2026
@eos87 eos87 requested a review from Copilot April 16, 2026 08:58

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.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@eos87 eos87 force-pushed the hg/i18n-translation-agent-skill branch from 5b1cdf6 to dfbea5d Compare April 16, 2026 09:16
@eos87 eos87 closed this Apr 16, 2026
@eos87 eos87 deleted the copilot/update-spanish-translations branch April 16, 2026 09:17
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.

3 participants