From ce456db4b778a23e02895fe798027cf9720e49f6 Mon Sep 17 00:00:00 2001 From: Kshitij Sobti Date: Mon, 21 Feb 2022 14:38:01 +0000 Subject: [PATCH] fix: discussion configuration not saved to course for new provider (#29863) This fixes an issue where updating settings for the new discussion provider using the API doesn't save them to the course. --- openedx/core/djangoapps/discussions/serializers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openedx/core/djangoapps/discussions/serializers.py b/openedx/core/djangoapps/discussions/serializers.py index 814e853ba8..874f5e62d2 100644 --- a/openedx/core/djangoapps/discussions/serializers.py +++ b/openedx/core/djangoapps/discussions/serializers.py @@ -295,7 +295,7 @@ class DiscussionsConfigurationSerializer(serializers.ModelSerializer): plugin_configuration = validated_data.pop('plugin_configuration', {}) updated_provider_type = validated_data.get('provider_type') or instance.provider_type - if updated_provider_type == Provider.LEGACY: + if updated_provider_type in [Provider.LEGACY, Provider.OPEN_EDX]: legacy_settings = LegacySettingsSerializer( self._get_course(), context={