feat!: remove all access to Old Mongo courses - #30172
Conversation
Change has_access to deny 'load' support for Old Mongo courses. This is in service of dropping support for these ancient courses and removing legacy code that they rely on. DEPR-58
|
Hmm, the same test failure didn't show up. And I can't reproduce locally either. It might have just been a flaky test and I over-reacted? I will try landing again. |
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
|
EdX Release Notice: This PR may have caused e2e tests to fail on Stage. If you're a member of the edX org, please visit #e2e-troubleshooting on Slack to help diagnose the cause of these failures. Otherwise, it is the reviewer's responsibility. E2E tests have failed. https://gocd.tools.edx.org/go/tab/pipeline/history/deploy_to_stage |
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
|
EdX Release Notice: This PR has been deployed to the production environment. |
LMS Courseware access to Old Mongo courses was already removed in fc8601d (#30172). This commit makes direct links to the other tabs (progress, instructor dashboard, discussion, static tabs) fail with a 404 error on Old Mongo courses. Upcoming work to remove parent/child relationships from the Old Mongo Modulestore would have broken these pages anyway.
LMS Courseware access to Old Mongo courses was already removed in fc8601d (#30172). This commit makes direct links to the other tabs (progress, instructor dashboard, discussion, static tabs) fail with a 404 error on Old Mongo courses. Upcoming work to remove parent/child relationships from the Old Mongo Modulestore would have broken these pages anyway.
2 participants
Change has_access to deny 'load' support for Old Mongo courses.
This is in service of dropping support for these ancient courses and removing legacy code that they rely on.
DEPR-58
DEPR-123
Here's a screenshot of what the dashboard looks like after this change. You can see both the error message we show when you try to go to an Old Mongo course (you get redirected to dashboard with this mesage). And you can see how an Old Mongo course card shows up - it's still there, but without any links.
This previously landed in #29848 but got reverted because it accidentally broke some tests.