fixup! fix: adjust studio_grading_url to use the correct URL

Author: dwong2708

lms/djangoapps/instructor/tests/test_api_v2.py

@@ -166,7 +166,6 @@ def test_get_course_metadata_as_instructor(self):
 
         # Verify other metadata fields
         assert 'num_sections' in data
-        assert 'tabs' in data
         assert 'grade_cutoffs' in data
         assert 'course_errors' in data
         assert 'studio_url' in data
@@ -180,6 +179,7 @@ def test_get_course_metadata_as_instructor(self):
         assert data['studio_grading_url'] == f'http://localhost:2001/authoring/course/{self.course.id}/settings/grading'
         assert data['admin_console_url'] == 'http://localhost:2025/admin-console/authz'
 
+    @override_settings(ADMIN_CONSOLE_MICROFRONTEND_URL='http://localhost:2025/admin-console')
     def test_admin_console_url_requires_instructor_access(self):
         """
         Test that the admin console URL is only available to users with instructor access.

lms/djangoapps/instructor/views/api_v2.py

@@ -206,7 +206,7 @@ def get(self, request, course_id):
                 "grade_cutoffs": "A is 0.9, B is 0.8, C is 0.7, D is 0.6",
                 "course_errors": [],
                 "studio_url": "https://studio.example.com/course/course-v1:edX+DemoX+2024",
-                "admin_console_url": "http://apps.local.openedx.io:2025/admin-console/authz",
+                "admin_console_url": "http://apps.local.openedx.io:2025/admin-console/authz",  # May be null if user does not have access
                 "permissions": {
                     "admin": false,
                     "instructor": true,

lms/djangoapps/instructor/views/serializers_v2.py

@@ -458,8 +458,7 @@ def get_gradebook_url(self, data):
     def get_studio_grading_url(self, data):
         """Get Studio MFE grading settings URL for the course."""
         course_key = data['course'].id
-        mfe_base_url = configuration_helpers.get_value_for_org(
-            course_key.org,
+        mfe_base_url = configuration_helpers.get_value(
             'MFE_CONFIG',
             getattr(settings, 'MFE_CONFIG', None)
         )
@@ -470,11 +469,9 @@ def get_studio_grading_url(self, data):
 
     def get_admin_console_url(self, data):
         """Get admin console URL (requires instructor access and MFE configuration, null if not accessible)."""
-        course_key = data['course'].id
         request = data['request']
         has_instructor_access = has_access(request.user, 'instructor', data['course'])
-        mfe_base_url = configuration_helpers.get_value_for_org(
-            course_key.org,
+        mfe_base_url = configuration_helpers.get_value(
             'ADMIN_CONSOLE_MICROFRONTEND_URL',
             getattr(settings, 'ADMIN_CONSOLE_MICROFRONTEND_URL', None)
         )