Skip to content

build: add support for additional translation sources - #126

Merged
arbrandes merged 2 commits into
openedx:2.xfrom
alenkadev:alenka/switching-to-local-translations
Aug 12, 2026
Merged

build: add support for additional translation sources#126
arbrandes merged 2 commits into
openedx:2.xfrom
alenkadev:alenka/switching-to-local-translations

Conversation

@alenkadev

Copy link
Copy Markdown

Title: Switch to using local translations (AU-2765)

Description:
Declares ATLAS_EXTRA_SOURCES, ATLAS_EXTRA_INTL_IMPORTS, and ATLAS_OPTIONS as overridable Makefile variables (?=, default empty).
Appends $(ATLAS_EXTRA_SOURCES) to the atlas pull command in pull_translations, so extra translations/: entries can be pulled alongside the existing ones.
Appends (ATLAS_EXTRA_INTL_IMPORTS) to the $(intl_imports) call, so any extra sources pulled above are also registered/imported

jira-link : https://2u-internal.atlassian.net/browse/LP-330

@alenkadev
alenkadev requested a review from arbrandes August 10, 2026 10:29
@arbrandes arbrandes changed the title feat: add support for additional translation sources and imports in M… build: add support for additional translation sources Aug 11, 2026

@arbrandes arbrandes 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.

Can you please change the commit prefix from feat: to build:? I don't think this warrants a new minor version of the plugin.

If, on the other hand, you think this warrants a new NPM release, then a fix: would be ok.

Comment thread Makefile Outdated
@alenkadev
alenkadev force-pushed the alenka/switching-to-local-translations branch from 155d8b2 to 199f737 Compare August 12, 2026 05:52

@arbrandes arbrandes 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.

It seems you missed a spot.

Comment thread Makefile Outdated
Co-authored-by: Adolfo R. Brandes <arbrandes@arbrand.es>
@arbrandes
arbrandes merged commit d85e047 into openedx:2.x Aug 12, 2026
4 checks passed
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.

2 participants