-
Notifications
You must be signed in to change notification settings - Fork 1
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
perf: sumac release support AP-1582
#74
Conversation
@BryanttV has this been tested in a remote env? |
Hi @magajh, Yes, these changes were tested in our Sumac remote environment. |
c3b1a75
to
18cdd64
Compare
18cdd64
to
049c939
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this! I haven't had time to replicate the test cases, but if we are sure this works in remote then LGTM
@BryanttV have we tested that the integration tests for Redwood are successful? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Description
This PR adds support for the Open edX Sumac release.
These were the changes made:
edx-platform
Sumac release.Testing instructions
Create an environment with Sumac release, you can use Tutor or TVM.
Install and enable tutor-contrib-picasso.
Install and enable tutor-mfe.
Add the following settings in your
config.yml
:Clone the test themes folder with
tutor picasso enable-themes
Add the testing themes running and init the environment with
tutor dev launch
Create a new tenant and add this settings to the tenant configs:
Go to your admin panel and edit the
learner_home_mfe.enabled
Waffle Flag with the propertyEveryone
inNo
to disable the Authn MFE.Run the test cases. In this file, you can find more information about the plugin configuration.
Related Issues