Skip to content

Commit e5cffc6

Browse files
feanilclaude
andcommitted
refactor: migrate STUDIO_REQUEST_EMAIL off FEATURES-as-dict
The v3 studio-home view still read the flag via `settings.FEATURES.get('STUDIO_REQUEST_EMAIL', '')`. The earlier STUDIO_REQUEST_EMAIL migration updated the v1 view (contentstore/rest_api/v1/views/home.py) and course_creators/admin.py to flat `settings.STUDIO_REQUEST_EMAIL`, but missed this duplicated v3 copy. The FeaturesProxy bridge kept it working, so CI stayed green. Read the flag off flat settings with `settings.STUDIO_REQUEST_EMAIL` (defined in cms/envs/common.py, default `''` — matching the prior `.get` default). Tests already override it flat via `@override_settings(..., STUDIO_REQUEST_EMAIL=...)`. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent b5622e7 commit e5cffc6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • cms/djangoapps/contentstore/rest_api/v3/views

cms/djangoapps/contentstore/rest_api/v3/views/home.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ def list(self, request: Request):
122122
),
123123
'studio_name': settings.STUDIO_NAME,
124124
'studio_short_name': settings.STUDIO_SHORT_NAME,
125-
'studio_request_email': settings.FEATURES.get('STUDIO_REQUEST_EMAIL', ''),
125+
'studio_request_email': settings.STUDIO_REQUEST_EMAIL,
126126
'tech_support_email': settings.TECH_SUPPORT_EMAIL,
127127
'platform_name': settings.PLATFORM_NAME,
128128
'user_is_active': request.user.is_active,

0 commit comments

Comments
 (0)