Skip to content

Commit fc38e4b

Browse files
AgrendalathCopilot
andcommitted
fix: update the Discussion course app on import
This fixes inconsistent Studio behavior in e1a0145. Co-authored-by: Copilot <copilot@github.com>
1 parent 42e22c1 commit fc38e4b

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

cms/djangoapps/contentstore/tasks.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@
6161
from common.djangoapps.util.monitoring import monitor_import_failure
6262
from openedx.core.djangoapps.content.learning_sequences.api import key_supports_outlines
6363
from openedx.core.djangoapps.content_libraries import api as v2contentlib_api
64+
from openedx.core.djangoapps.course_apps.api import set_course_app_enabled
6465
from openedx.core.djangoapps.course_apps.toggles import exams_ida_enabled
6566
from openedx.core.djangoapps.discussions.config.waffle import ENABLE_NEW_STRUCTURE_DISCUSSIONS
6667
from openedx.core.djangoapps.discussions.models import DiscussionsConfiguration, Provider
@@ -474,12 +475,14 @@ def sync_discussion_settings(course_key, user):
474475
if settings_key in discussion_settings:
475476
setattr(discussion_config, attr_name, discussion_settings[settings_key])
476477

478+
discussion_config.save()
479+
477480
# This part is no longer needed in Verawood.
478481
for tab in course.tabs:
479482
if tab.tab_id == "discussion":
480-
discussion_config.enabled = not tab.is_hidden
483+
set_course_app_enabled(course_key, "discussion", not tab.is_hidden, user)
481484
break
482-
discussion_config.save()
485+
483486
LOGGER.info(f'Course import {course.id}: DiscussionsConfiguration synced as per course')
484487
except Exception as exc: # pylint: disable=broad-except
485488
LOGGER.info(f'Course import {course.id}: DiscussionsConfiguration sync failed: {exc}')

0 commit comments

Comments
 (0)