feat: Adds disable_progress_graph attribute to the returned course_me… - #34324
Conversation
openedx-webhooks
added
the
open-source-contribution
|
Thanks for the pull request, @dyudyunov! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
mphilbrick211
added
the
backport
|
Note: the related Learning MFE PR was merged openedx/frontend-app-learning#1313 |
|
@mphilbrick211 can someone review this? |
Thanks for flagging, @dyudyunov - will look into getting a reviewer. Thanks for your patience! |
|
Sorry for flagging this one again |
|
Hi @dyudyunov! I'm sorry for the delay. I'm trying to get a core contributor to look at this for you. |
|
@dyudyunov Please rebase the branch on the latest |
…tadata response Currently, openedx/frontend-app-authoring#517 faces an issue when the progress graph toggle is enabled/disabled but the settings are not respected, the disable_progress_graph attribute will allow the frontend-app-learning repo to use this attribute to respect the settings authored from frontend-app-course-authoring and ultimately fix openedx/frontend-app-authoring#517.
7ac4adc to
2eb82ad
Compare
|
@dyudyunov @mphilbrick211 Verified the fix on test env and going to merge this. 👍 |
|
@dyudyunov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
5 participants
Description
This is a backport from the master branch: #33308
Fix an issue when the Progress Graph toggle change in the studio has no effect on the Learning MFE.
Related backport for the Learning MFE: openedx/frontend-app-learning#1313
Please check the original PRs for more details.