-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
fix: update course discussion config before course load #35219
Conversation
fix: update course discussion config before course load fix: updated unit tests fix: resolved content issue in renamed notification
be4caaa
to
a8fd50b
Compare
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
@AhtishamShahid it appears that this PR is causing issues when trying to load large courses. The api returns a 504 error and prevents the page from loading. I am going to revert the PR. Please reach out for more details |
…)" This reverts commit 5c09424.
…edx#35219)" (openedx#35349) This reverts commit 5c09424.
Description
Ensure to update course discussions settings before course is loaded cms , sometimes newly created courses are not synced to real discussions settings which causes false information for the user.
Ticket
https://2u-internal.atlassian.net/browse/INF-1469