From 54044232b8d7bf48554cbacd3bbf78465ce7d009 Mon Sep 17 00:00:00 2001
From: Robert Raposa <rraposa@edx.org>
Date: Fri, 11 Dec 2020 11:16:19 -0500
Subject: [PATCH] update FEATURES toggle annotation names
Update toggle_name annotations from "XXX"
to "FEATURES['XXX']" to match the name
used by the toggle state endpoint so they
don't seem like two different toggles.
---
cms/envs/common.py | 10 +++----
lms/envs/common.py | 66 +++++++++++++++++++++++-----------------------
2 files changed, 38 insertions(+), 38 deletions(-)
diff --git a/cms/envs/common.py b/cms/envs/common.py
index 7dd9d6d6ebd..61c982b83fe 100644
--- a/cms/envs/common.py
+++ b/cms/envs/common.py
@@ -340,7 +340,7 @@ FEATURES = {
'ENABLE_ENROLLMENT_RESET': False,
'DISABLE_MOBILE_COURSE_AVAILABLE': False,
- # .. toggle_name: ENABLE_CHANGE_USER_PASSWORD_ADMIN
+ # .. toggle_name: FEATURES['ENABLE_CHANGE_USER_PASSWORD_ADMIN']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to enable changing a user password through django admin. This is disabled by
@@ -353,7 +353,7 @@ FEATURES = {
'ENABLE_CHANGE_USER_PASSWORD_ADMIN': False,
### ORA Feature Flags ###
- # .. toggle_name: ENABLE_ORA_ALL_FILE_URLS
+ # .. toggle_name: FEATURES['ENABLE_ORA_ALL_FILE_URLS']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: A "work-around" feature toggle meant to help in cases where some file uploads are not
@@ -366,7 +366,7 @@ FEATURES = {
# .. toggle_warnings: This temporary feature toggle does not have a target removal date.
'ENABLE_ORA_ALL_FILE_URLS': False,
- # .. toggle_name: ENABLE_ORA_USER_STATE_UPLOAD_DATA
+ # .. toggle_name: FEATURES['ENABLE_ORA_USER_STATE_UPLOAD_DATA']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: A "work-around" feature toggle meant to help in cases where some file uploads are not
@@ -378,7 +378,7 @@ FEATURES = {
# .. toggle_warnings: This temporary feature toggle does not have a target removal date.
'ENABLE_ORA_USER_STATE_UPLOAD_DATA': False,
- # .. toggle_name: DEPRECATE_OLD_COURSE_KEYS_IN_STUDIO
+ # .. toggle_name: FEATURES['DEPRECATE_OLD_COURSE_KEYS_IN_STUDIO']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Warn about removing support for deprecated course keys.
@@ -394,7 +394,7 @@ FEATURES = {
# .. toggle_tickets: https://openedx.atlassian.net/browse/DEPR-58
'DEPRECATE_OLD_COURSE_KEYS_IN_STUDIO': True,
- # .. toggle_name: ENABLE_LIBRARY_AUTHORING_MICROFRONTEND
+ # .. toggle_name: FEATURES['ENABLE_LIBRARY_AUTHORING_MICROFRONTEND']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to enable the Library Authoring MFE
diff --git a/lms/envs/common.py b/lms/envs/common.py
index ec97c9cff92..dc6c7506a04 100644
--- a/lms/envs/common.py
+++ b/lms/envs/common.py
@@ -83,7 +83,7 @@ IDA_LOGOUT_URI_LIST = []
# Features
FEATURES = {
- # .. toggle_name: DISPLAY_DEBUG_INFO_TO_STAFF
+ # .. toggle_name: FEATURES['DISPLAY_DEBUG_INFO_TO_STAFF']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Add a "Staff Debug" button to course modules for debugging
@@ -98,7 +98,7 @@ FEATURES = {
'REROUTE_ACTIVATION_EMAIL': False, # nonempty string = address for all activation emails
- # .. toggle_name: DISABLE_START_DATES
+ # .. toggle_name: FEATURES['DISABLE_START_DATES']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: When True, all courses will be active, regardless of start
@@ -116,7 +116,7 @@ FEATURES = {
# in sync with the corresponding ones in cms/envs/common.py
'ENABLE_DISCUSSION_SERVICE': True,
- # .. toggle_name: ENABLE_TEXTBOOK
+ # .. toggle_name: FEATURES['ENABLE_TEXTBOOK']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Add PDF and HTML textbook tabs to the courseware.
@@ -131,7 +131,7 @@ FEATURES = {
# this should remain off in production until digest notifications are online.
'ENABLE_DISCUSSION_HOME_PANEL': False,
- # .. toggle_name: ENABLE_DISCUSSION_EMAIL_DIGEST
+ # .. toggle_name: FEATURES['ENABLE_DISCUSSION_EMAIL_DIGEST']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set this to True if you want the discussion digest emails
@@ -146,7 +146,7 @@ FEATURES = {
# .. toggle_tickets: https://github.com/edx/edx-platform/pull/4891
'ENABLE_DISCUSSION_EMAIL_DIGEST': False,
- # .. toggle_name: ENABLE_UNICODE_USERNAME
+ # .. toggle_name: FEATURES['ENABLE_UNICODE_USERNAME']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set this to True to allow unicode characters in username. Enabling this will also
@@ -162,7 +162,7 @@ FEATURES = {
'ENABLE_DJANGO_ADMIN_SITE': True, # set true to enable django's admin site, even on prod (e.g. for course ops)
'ENABLE_LMS_MIGRATION': False,
- # .. toggle_name: ENABLE_MASQUERADE
+ # .. toggle_name: FEATURES['ENABLE_MASQUERADE']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: None
@@ -173,7 +173,7 @@ FEATURES = {
# .. toggle_tickets: None
'ENABLE_MASQUERADE': True,
- # .. toggle_name: ENABLE_SYSADMIN_DASHBOARD
+ # .. toggle_name: FEATURES['ENABLE_SYSADMIN_DASHBOARD']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: enables dashboard at /syadmin/ for django staff, for seeing overview of system status, for
@@ -187,7 +187,7 @@ FEATURES = {
'DISABLE_LOGIN_BUTTON': False, # used in systems where login is automatic, eg MIT SSL
- # .. toggle_name: ENABLE_OAUTH2_PROVIDER
+ # .. toggle_name: FEATURES['ENABLE_OAUTH2_PROVIDER']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Enable this feature to allow this Open edX platform to be an OAuth2 authentication
@@ -199,7 +199,7 @@ FEATURES = {
# .. toggle_tickets: None
'ENABLE_OAUTH2_PROVIDER': False,
- # .. toggle_name: ENABLE_XBLOCK_VIEW_ENDPOINT
+ # .. toggle_name: FEATURES['ENABLE_XBLOCK_VIEW_ENDPOINT']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Enable an API endpoint, named "xblock_view", to serve rendered XBlock views. This might be
@@ -257,7 +257,7 @@ FEATURES = {
# Toggle to enable certificates of courses on dashboard
'ENABLE_VERIFIED_CERTIFICATES': False,
- # .. toggle_name: DISABLE_HONOR_CERTIFICATES
+ # .. toggle_name: FEATURES['DISABLE_HONOR_CERTIFICATES']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to disable honor certificates. Typically used when your installation only
@@ -274,7 +274,7 @@ FEATURES = {
# for acceptance and load testing
'AUTOMATIC_AUTH_FOR_TESTING': False,
- # .. toggle_name: RESTRICT_AUTOMATIC_AUTH
+ # .. toggle_name: FEATURES['RESTRICT_AUTOMATIC_AUTH']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Prevent auto auth from creating superusers or modifying existing users. Auto auth is a
@@ -288,7 +288,7 @@ FEATURES = {
# .. toggle_tickets: https://openedx.atlassian.net/browse/TE-2545
'RESTRICT_AUTOMATIC_AUTH': True,
- # .. toggle_name: ENABLE_LOGIN_MICROFRONTEND
+ # .. toggle_name: FEATURES['ENABLE_LOGIN_MICROFRONTEND']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Enable the login micro frontend.
@@ -299,7 +299,7 @@ FEATURES = {
# .. toggle_tickets: None
'ENABLE_LOGIN_MICROFRONTEND': False,
- # .. toggle_name: SKIP_EMAIL_VALIDATION
+ # .. toggle_name: FEATURES['SKIP_EMAIL_VALIDATION']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Turn this on to skip sending emails for user validation.
@@ -311,7 +311,7 @@ FEATURES = {
# .. toggle_tickets: None
'SKIP_EMAIL_VALIDATION': False,
- # .. toggle_name: ENABLE_COSMETIC_DISPLAY_PRICE
+ # .. toggle_name: FEATURES['ENABLE_COSMETIC_DISPLAY_PRICE']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Enable the display of "cosmetic_display_price", set in a course advanced settings. This
@@ -351,7 +351,7 @@ FEATURES = {
# defaults, so that we maintain current behavior
'ALLOW_WIKI_ROOT_ACCESS': True,
- # .. toggle_name: ENABLE_THIRD_PARTY_AUTH
+ # .. toggle_name: FEATURES['ENABLE_THIRD_PARTY_AUTH']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Turn on third-party auth. Disabled for now because full mplementations are not yet
@@ -365,7 +365,7 @@ FEATURES = {
# .. toggle_tickets: None
'ENABLE_THIRD_PARTY_AUTH': False,
- # .. toggle_name: ENABLE_MKTG_SITE
+ # .. toggle_name: FEATURES['ENABLE_MKTG_SITE']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Toggle to enable alternate urls for marketing links.
@@ -385,7 +385,7 @@ FEATURES = {
# False to not redirect the user
'ALWAYS_REDIRECT_HOMEPAGE_TO_DASHBOARD_FOR_AUTHENTICATED_USER': True,
- # .. toggle_name: ENABLE_COURSE_SORTING_BY_START_DATE
+ # .. toggle_name: FEATURES['ENABLE_COURSE_SORTING_BY_START_DATE']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: When a user goes to the homepage ('/') the user sees the courses listed in the
@@ -398,7 +398,7 @@ FEATURES = {
# .. toggle_tickets: https://github.com/edx/edx-platform/pull/7548
'ENABLE_COURSE_SORTING_BY_START_DATE': True,
- # .. toggle_name: ENABLE_COURSE_HOME_REDIRECT
+ # .. toggle_name: FEATURES['ENABLE_COURSE_HOME_REDIRECT']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: When enabled, along with the ENABLE_MKTG_SITE feature toggle, users who attempt to access a
@@ -415,7 +415,7 @@ FEATURES = {
# ENABLE_OAUTH2_PROVIDER to True
'ENABLE_MOBILE_REST_API': False,
- # .. toggle_name: ENABLE_COMBINED_LOGIN_REGISTRATION_FOOTER
+ # .. toggle_name: FEATURES['ENABLE_COMBINED_LOGIN_REGISTRATION_FOOTER']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Display the standard footer in the login page. This feature can be overridden by a site-
@@ -462,7 +462,7 @@ FEATURES = {
# log all information from cybersource callbacks
'LOG_POSTPAY_CALLBACKS': True,
- # .. toggle_name: LICENSING
+ # .. toggle_name: FEATURES['LICENSING']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Toggle platform-wide course licensing. The course.license attribute is then used to append
@@ -477,7 +477,7 @@ FEATURES = {
# Certificates Web/HTML Views
'CERTIFICATES_HTML_VIEW': False,
- # .. toggle_name: ENABLE_COURSE_DISCOVERY
+ # .. toggle_name: FEATURES['ENABLE_COURSE_DISCOVERY']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Add a course search widget to the LMS for searching courses. When this is enabled, the
@@ -505,7 +505,7 @@ FEATURES = {
# How many seconds to show the bumper again, default is 7 days:
'SHOW_BUMPER_PERIODICITY': 7 * 24 * 3600,
- # .. toggle_name: ENABLE_SPECIAL_EXAMS
+ # .. toggle_name: FEATURES['ENABLE_SPECIAL_EXAMS']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Enable to use special exams, aka timed and proctored exams.
@@ -533,7 +533,7 @@ FEATURES = {
# Show the language selector in the footer
'SHOW_FOOTER_LANGUAGE_SELECTOR': False,
- # .. toggle_name: ENABLE_CSMH_EXTENDED
+ # .. toggle_name: FEATURES['ENABLE_CSMH_EXTENDED']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Write Courseware Student Module History (CSMH) to the extended table: this logs all
@@ -556,7 +556,7 @@ FEATURES = {
# Set this to False to facilitate cleaning up invalid xml from your modulestore.
'ENABLE_XBLOCK_XML_VALIDATION': True,
- # .. toggle_name: ALLOW_PUBLIC_ACCOUNT_CREATION
+ # .. toggle_name: FEATURES['ALLOW_PUBLIC_ACCOUNT_CREATION']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: Allow public account creation. If this is disabled, users will no longer have access to
@@ -594,7 +594,7 @@ FEATURES = {
# Whether HTML XBlocks/XModules return HTML content with the Course Blocks API student_view_data
'ENABLE_HTML_XBLOCK_STUDENT_VIEW_DATA': False,
- # .. toggle_name: ENABLE_CHANGE_USER_PASSWORD_ADMIN
+ # .. toggle_name: FEATURES['ENABLE_CHANGE_USER_PASSWORD_ADMIN']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Whether to send an email for failed password reset attempts or not. This happens when a
@@ -618,7 +618,7 @@ FEATURES = {
'ENABLE_ENROLLMENT_RESET': False,
'DISABLE_MOBILE_COURSE_AVAILABLE': False,
- # .. toggle_name: ENABLE_CHANGE_USER_PASSWORD_ADMIN
+ # .. toggle_name: FEATURES['ENABLE_CHANGE_USER_PASSWORD_ADMIN']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to enable changing a user password through django admin. This is disabled by
@@ -630,7 +630,7 @@ FEATURES = {
# .. toggle_tickets: 'https://github.com/edx/edx-platform/pull/21616'
'ENABLE_CHANGE_USER_PASSWORD_ADMIN': False,
- # .. toggle_name: ENABLE_COURSEWARE_MICROFRONTEND
+ # .. toggle_name: FEATURES['ENABLE_COURSEWARE_MICROFRONTEND']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to enable the Courseware MFE at the platform level for global staff (see
@@ -643,7 +643,7 @@ FEATURES = {
# rollout.
'ENABLE_COURSEWARE_MICROFRONTEND': False,
- # .. toggle_name: ENABLE_LOGISTRATION_MICROFRONTEND
+ # .. toggle_name: FEATURES['ENABLE_LOGISTRATION_MICROFRONTEND']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Supports staged rollout of a new micro-frontend-based implementation of the logistration.
@@ -656,7 +656,7 @@ FEATURES = {
'ENABLE_LOGISTRATION_MICROFRONTEND': False,
### ORA Feature Flags ###
- # .. toggle_name: ENABLE_ORA_ALL_FILE_URLS
+ # .. toggle_name: FEATURES['ENABLE_ORA_ALL_FILE_URLS']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: A "work-around" feature toggle meant to help in cases where some file uploads are not
@@ -669,7 +669,7 @@ FEATURES = {
# .. toggle_warnings: This temporary feature toggle does not have a target removal date.
'ENABLE_ORA_ALL_FILE_URLS': False,
- # .. toggle_name: ENABLE_ORA_USER_STATE_UPLOAD_DATA
+ # .. toggle_name: FEATURES['ENABLE_ORA_USER_STATE_UPLOAD_DATA']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: A "work-around" feature toggle meant to help in cases where some file uploads are not
@@ -681,7 +681,7 @@ FEATURES = {
# .. toggle_warnings: This temporary feature toggle does not have a target removal date.
'ENABLE_ORA_USER_STATE_UPLOAD_DATA': False,
- # .. toggle_name: ENABLE_ORA_USERNAMES_ON_DATA_EXPORT
+ # .. toggle_name: FEATURES['ENABLE_ORA_USERNAMES_ON_DATA_EXPORT']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to add deanonymized usernames to ORA data
@@ -693,7 +693,7 @@ FEATURES = {
# .. toggle_warnings: This temporary feature toggle does not have a target removal date.
'ENABLE_ORA_USERNAMES_ON_DATA_EXPORT': False,
- # .. toggle_name: ENABLE_COURSE_ASSESSMENT_GRADE_CHANGE_SIGNAL
+ # .. toggle_name: FEATURES['ENABLE_COURSE_ASSESSMENT_GRADE_CHANGE_SIGNAL']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: False
# .. toggle_description: Set to True to start sending signals for assessment level grade updates. Notably, the only
@@ -2954,7 +2954,7 @@ DISABLE_ACCOUNT_ACTIVATION_REQUIREMENT_SWITCH = "verify_student_disable_account_
################ Enable credit eligibility feature ####################
ENABLE_CREDIT_ELIGIBILITY = True
-# .. toggle_name: ENABLE_CREDIT_ELIGIBILITY
+# .. toggle_name: FEATURES['ENABLE_CREDIT_ELIGIBILITY']
# .. toggle_implementation: DjangoSetting
# .. toggle_default: True
# .. toggle_description: When enabled, it is possible to define a credit eligibility criteria in the CMS. A "Credit
--
GitLab