From 3cbc03202cf1ac5585e8b2fbc14ecc513e211812 Mon Sep 17 00:00:00 2001 From: Renzo Lucioni Date: Fri, 25 Mar 2016 15:55:58 -0400 Subject: [PATCH] Hide chrome and default_tab from advanced settings It doesn't make sense to list these LMS block attributes in course-level advanced settings. Doing so is confusing to course teams. --- cms/djangoapps/models/settings/course_metadata.py | 4 +++- common/test/acceptance/pages/studio/settings_advanced.py | 2 -- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cms/djangoapps/models/settings/course_metadata.py b/cms/djangoapps/models/settings/course_metadata.py index ad253e1506..5859205317 100644 --- a/cms/djangoapps/models/settings/course_metadata.py +++ b/cms/djangoapps/models/settings/course_metadata.py @@ -50,7 +50,9 @@ class CourseMetadata(object): 'is_time_limited', 'is_practice_exam', 'exam_review_rules', - 'self_paced' + 'self_paced', + 'chrome', + 'default_tab', ] @classmethod diff --git a/common/test/acceptance/pages/studio/settings_advanced.py b/common/test/acceptance/pages/studio/settings_advanced.py index 10b3939c08..e209207189 100644 --- a/common/test/acceptance/pages/studio/settings_advanced.py +++ b/common/test/acceptance/pages/studio/settings_advanced.py @@ -184,9 +184,7 @@ class AdvancedSettingsPage(CoursePage): 'display_coursenumber', 'display_organization', 'catalog_visibility', - 'chrome', 'days_early_for_beta', - 'default_tab', 'disable_progress_graph', 'discussion_blackouts', 'discussion_sort_alpha',