fix: missing discussions provider

open-craft · Oct 25, 2024 · 0e4a74a · 0e4a74a
1 parent b75c2db
commit 0e4a74a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/cms/djangoapps/contentstore/utils.py b/cms/djangoapps/contentstore/utils.py
@@ -1085,6 +1085,7 @@ def update_course_discussions_settings(course):
     """
     provider = DiscussionsConfiguration.get(context_key=course.id).provider_type
     store = modulestore()
+    course.discussions_settings['provider'] = provider
     course.discussions_settings['provider_type'] = provider
     store.update_item(course, course.published_by)
 

diff --git a/openedx/core/djangoapps/discussions/tasks.py b/openedx/core/djangoapps/discussions/tasks.py
@@ -196,7 +196,7 @@ def update_unit_discussion_state_from_discussion_blocks(course_key: CourseKey, u
     """
     store = modulestore()
     course = store.get_course(course_key)
-    provider = course.discussions_settings.get('provider', None)
+    provider = course.discussions_settings.get('provider', None) or course.discussions_settings.get('provider_type', None)
     # Only migrate to the new discussion provider if the current provider is the legacy provider.
     log.info(f"Current provider for {course_key} is {provider}")
     if provider is not None and provider != Provider.LEGACY and not force: