Merge pull request #20126 from edx/depr/mobile-video-outline-api

DEPR-4: Remove Mobile Video Outlines API

lms/djangoapps/mobile_api/users/serializers.py

@@ -73,11 +73,10 @@ class CourseOverviewField(serializers.RelatedField):
                 request=request,
             ) if course_overview.is_discussion_tab_enabled() else None,
 
-            'video_outline': reverse(
-                'video-summary-list',
-                kwargs={'api_version': api_version, 'course_id': course_id},
-                request=request,
-            ),
+            # This is an old API that was removed as part of DEPR-4. We keep the
+            # field present in case API parsers expect it, but this API is now
+            # removed.
+            'video_outline': None,
         }
 
 

lms/djangoapps/mobile_api/users/tests.py

@@ -111,7 +111,6 @@ class TestUserEnrollmentApi(UrlResetMixin, MobileAPITestCase, MobileAuthUserTest
         self.assertIn('courses/{}/about'.format(self.course.id), found_course['course_about'])
         self.assertIn('course_info/{}/updates'.format(self.course.id), found_course['course_updates'])
         self.assertIn('course_info/{}/handouts'.format(self.course.id), found_course['course_handouts'])
-        self.assertIn('video_outlines/courses/{}'.format(self.course.id), found_course['video_outline'])
         self.assertEqual(found_course['id'], unicode(self.course.id))
         self.assertEqual(courses[0]['mode'], CourseMode.DEFAULT_MODE_SLUG)
         self.assertEqual(courses[0]['course']['subscription_id'], self.course.clean_id(padding_char='_'))