From 39543896200c98d5e6bf7ba73e847c1cda089405 Mon Sep 17 00:00:00 2001 From: Kshitij Sobti Date: Thu, 3 Feb 2022 13:10:42 +0530 Subject: [PATCH] fix: discussion configuration not saved to course for new provider 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 814e853ba867..874f5e62d237 100644 --- a/openedx/core/djangoapps/discussions/serializers.py +++ b/openedx/core/djangoapps/discussions/serializers.py @@ -295,7 +295,7 @@ def _update_plugin_configuration( 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={