Merge pull request #169 from edx/dsjen/fix-course-ids

Fix course summaries query arg to use course_ids.
openedx-unsupported · May 22, 2017 · 5a85c45 · 5a85c45
2 parents 662acec + 32b502c
commit 5a85c45
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/analytics_data_api/v0/tests/views/test_course_summaries.py b/analytics_data_api/v0/tests/views/test_course_summaries.py
@@ -139,7 +139,7 @@ def test_empty_modes(self, modes):
         [CourseSamples.course_ids[0], 'malformed-course-id'],
     )
     def test_bad_course_id(self, course_ids):
-        response = self.authenticated_get(self.path(ids=course_ids))
+        response = self.authenticated_get(self.path(course_ids=course_ids))
         self.verify_bad_course_id(response)
 
     def test_collapse_upcoming(self):

diff --git a/analytics_data_api/v0/views/course_summaries.py b/analytics_data_api/v0/views/course_summaries.py
@@ -15,7 +15,7 @@ class CourseSummariesView(APIListView):
 
     **Example Request**
 
-        GET /api/v0/course_summaries/?ids={course_id},{course_id}
+        GET /api/v0/course_summaries/?course_ids={course_id},{course_id}
 
     **Response Values**
 
@@ -63,7 +63,7 @@ def get(self, request, *args, **kwargs):
         programs = split_query_argument(query_params.get('programs'))
         if not programs:
             self.always_exclude = self.always_exclude + ['programs']
-        self.ids = split_query_argument(query_params.get('ids'))
+        self.ids = split_query_argument(query_params.get('course_ids'))
         self.verify_ids()
         response = super(CourseSummariesView, self).get(request, *args, **kwargs)
         return response