Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: switch to upstream assets.sh #28

Closed
wants to merge 3 commits into from

Conversation

kdmccormick
Copy link
Owner

@kdmccormick kdmccormick force-pushed the kdmccormick/assets-sh branch 2 times, most recently from 47ef233 to 4da3617 Compare April 14, 2023 20:07
When enabled, this flag tells Tutor's openedx-assets script
(which processes assets using Python and edx-platform's Paver tools)
to instead proxy through to edx-platform's new asset processing system
(`scripts/build-assets.sh` and `./manage.py collectstatic`).

Also, when enabled, openedx-assets will print out the shell command
that could have been used to directly invoke the new asset processing
system. In the future, openedx-assets will be deprecated, and the
direct shell command will be the canonical way of compiling assets.

Part of: openedx/edx-platform#31798
ADR: https://github.com/openedx/edx-platform/blob/master/docs/decisions/0017-reimplement-asset-processing.rst
@kdmccormick kdmccormick force-pushed the kdmccormick/assets-sh branch from 4da3617 to 43b7891 Compare April 14, 2023 20:39
@kdmccormick
Copy link
Owner Author

closed in favor of #31

@kdmccormick kdmccormick deleted the kdmccormick/assets-sh branch July 25, 2023 20:03
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.

1 participant