fix: discussion configuration not saved to course for new provider [BD-38] [TNL-8622] #29863
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
openedx-webhooks
added
blended
|
Thanks for the pull request, @xitij2000! I've created BLENDED-1085 to keep track of it in Jira. More details are on the BD-38 project page. When this pull request is ready, tag your edX technical lead. |
xitij2000
force-pushed
the
kshitij/tnl-8622/fix-provider-bug
branch
2 times, most recently
from
d567ab7 to
7f4c872
Compare
This fixes an issue where updating settings for the new discussion provider using the API doesn't save them to the course.
xitij2000
force-pushed
the
kshitij/tnl-8622/fix-provider-bug
branch
from
7f4c872 to
3954389
Compare
Contributor
|
👍
|
asadazam93
approved these changes
Feb 21, 2022
|
@xitij2000 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Contributor
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
Contributor
|
EdX Release Notice: This PR has been deployed to the production environment. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Description
Fix discussion configuration API to save changes made to the new provider to the course.
Supporting information
Testing instructions