Skip to content

fix: translation make commands#236

Merged
irtazaakram merged 1 commit intomainfrom
fix-translations
Apr 21, 2026
Merged

fix: translation make commands#236
irtazaakram merged 1 commit intomainfrom
fix-translations

Conversation

@irtazaakram
Copy link
Copy Markdown
Member

@irtazaakram irtazaakram commented Apr 20, 2026

Changes

  • Fix extract_translations: drop --merge-po-files flag that crashes
    i18n-tools when an XBlock has no JS strings (html, lti, poll); merge
    manually with an existence check instead. Fix EXTRACT_DIR relative path
    so the django.potext.po rename actually happens.
  • Fix compile_translations: iterate over each XBlock instead of running
    once from the repo root.
  • Fix build_dummy_translations: add missing extract_translations
    dependency so validate_translations works from a clean checkout.
  • Fix XBLOCKS find: add -mindepth 2 to exclude the root-level combined
  • Add combined django.po output: extract_translations now merges all
    per-xblock text.po files into
    xblocks_contrib/conf/locale/en/LC_MESSAGES/django.po for the
    openedx-translations pipeline (OEP-58).
  • Fix .tx/config in all 8 XBlocks: correct the Transifex project from
    the non-existent p:xblocks to p:openedx-translations, and fix
    file_filter/source_file paths to match conf/locale/.
  • Update README and docs with OEP-58 workflow, all Make targets, and
    troubleshooting guide.

Testing

  • make extract_translations — produces text.po for all 8 XBlocks +
    combined django.po
  • make validate_translations — full pipeline passes from clean checkout
  • Simulated the openedx-translations GitHub Actions workflow locally
    (extract → compile → cleanup per-xblock dirs → find single EN_DIR →
    find django.po) — all steps pass

Related PR openedx/openedx-translations#70340
Most of the changes and Documentation is generated by Claude code.

@irtazaakram irtazaakram self-assigned this Apr 20, 2026
@irtazaakram irtazaakram moved this to 👀 In review in Aximprovements Team Apr 20, 2026
@irtazaakram irtazaakram merged commit ed847a5 into main Apr 21, 2026
13 checks passed
@irtazaakram irtazaakram deleted the fix-translations branch April 21, 2026 07:42
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Aximprovements Team Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants