share st-lutherx theme with edx-platform themes #7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For overriding the react js files we need to put our custom theme
st-lutherxinside theedx-platform / themes / st-lutherx.For dev environment, devops have updated the configuration to copy the themes in the
edly-edx-themesin theedx-platform / themes / st-lutherxto make the custom react js files work.For devstack we have a workaround to create an empty folder
st-lutherxinside theedx-platform / themesthe stop and up the devstack (with current changes), this will mount / sync thest-lutherxtheme../src/edly-edx-themes/st-lutherxwith the empty folderst-lutherxinside theedx-platform / themes.Also created an new docker make command
stop-extrato stop the extra containers which we don't need for Edly development environment.Compiling the
st-lutherxthemes:Note 1: Make sure now you have only one path
/edx/app/edxapp/edx-platform/themesin the config var "COMPREHENSIVE_THEME_DIRS" otherwise if we dont follow this step then we will get errors likeReactRenderermodule not found etcNote 2: Please take a backup of your
../lms.env.jsonbefore stopping and starting the devstack as the changes in../lms.env.jsonlost on devstack restart