Skip to content
This repository was archived by the owner on Aug 3, 2026. It is now read-only.

feat: optionally collect JS translations on deploy - #188

Merged
mtyaka merged 1 commit into
opencraft-release/lilac.2from
mtyaka/SE-5140-simple-theme-i18n-lilac
Apr 4, 2022
Merged

feat: optionally collect JS translations on deploy#188
mtyaka merged 1 commit into
opencraft-release/lilac.2from
mtyaka/SE-5140-simple-theme-i18n-lilac

Conversation

@mtyaka

@mtyaka mtyaka commented Mar 30, 2022

Copy link
Copy Markdown
Member

If the EDXAPP_COMPILE_JSI18N is enabled, this will invoke the compilejsi18n management command when deploying edxapp.

This is useful when you use custom JS translations for example via a theme.

This cherry-picks changes from openedx-unsupported#6708

If the EDXAPP_COMPILE_JSI18N is enabled, this will invoke the
compilejsi18n management command when deploying edxapp.

This is useful when you use custom JS translations for example via a
theme.
@mtyaka
mtyaka force-pushed the mtyaka/SE-5140-simple-theme-i18n-lilac branch from 5f708ab to 4bb4424 Compare March 30, 2022 09:10

@farhaanbukhsh farhaanbukhsh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

  • ✅ I tested the upstream changes.
  • ✅ I read through the code
  • ❌ I checked for accessibility issues
  • ❌ Includes documentation
  • ❌ I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@mtyaka

mtyaka commented Apr 4, 2022

Copy link
Copy Markdown
Member Author

Thanks @farhaanbukhsh!

@mtyaka
mtyaka merged commit 1dbf3ba into opencraft-release/lilac.2 Apr 4, 2022
@mtyaka
mtyaka deleted the mtyaka/SE-5140-simple-theme-i18n-lilac branch April 4, 2022 15:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants