From 6e336e2d391db71dccbfb8d74547957cb81f5539 Mon Sep 17 00:00:00 2001 From: Kshitij Sobti Date: Thu, 26 Jun 2025 15:57:41 +0530 Subject: [PATCH] fixup! fix: use a single 'provider_type' key for storing discussion provider type in course --- cms/djangoapps/contentstore/tasks.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cms/djangoapps/contentstore/tasks.py b/cms/djangoapps/contentstore/tasks.py index 4afd1cdcd5..fe8597db73 100644 --- a/cms/djangoapps/contentstore/tasks.py +++ b/cms/djangoapps/contentstore/tasks.py @@ -473,8 +473,8 @@ def sync_discussion_settings(course_key, user): if ( ENABLE_NEW_STRUCTURE_DISCUSSIONS.is_enabled() - and not course.discussions_settings['provider_type'] == Provider.OPEN_EDX - and not course.discussions_settings['provider'] == Provider.OPEN_EDX + and not course.discussions_settings.get('provider_type', None) == Provider.OPEN_EDX + and not course.discussions_settings.get('provider', None) == Provider.OPEN_EDX ): LOGGER.info(f"New structure is enabled, also updating {course_key} to use new provider") course.discussions_settings['enable_graded_units'] = False