feat: upgrading simple api to drf compatible.

openedx · Aug 8, 2024 · 28afb4b · 28afb4b
1 parent 4d12abd
commit 28afb4b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lms/djangoapps/instructor/views/api.py b/lms/djangoapps/instructor/views/api.py
@@ -105,7 +105,7 @@
 from lms.djangoapps.instructor_task.api_helper import AlreadyRunningError, QueueConnectionError
 from lms.djangoapps.instructor_task.data import InstructorTaskTypes
 from lms.djangoapps.instructor_task.models import ReportStore
-from lms.djangoapps.instructor.views.serializer import RoleNameSerializer, UserSerializer, UserAccessSerializer
+from lms.djangoapps.instructor.views.serializer import RoleNameSerializer, UserSerializer, AccessSerializer
 from openedx.core.djangoapps.content.course_overviews.models import CourseOverview
 from openedx.core.djangoapps.course_groups.cohorts import add_user_to_cohort, is_course_cohorted
 from openedx.core.djangoapps.course_groups.models import CourseUserGroup
@@ -996,7 +996,7 @@ class ModifyAccess(APIView):
     """
     permission_classes = (IsAuthenticated, permissions.InstructorPermission)
     permission_name = permissions.EDIT_COURSE_ACCESS
-    serializer_class = UserAccessSerializer
+    serializer_class = AccessSerializer
 
     @method_decorator(ensure_csrf_cookie)
     def post(self, request, course_id):
@@ -1006,7 +1006,7 @@ def post(self, request, course_id):
             request.user, 'instructor', course_id, depth=None
         )
 
-        serializer_data = UserAccessSerializer(data=request.data)
+        serializer_data = AccessSerializer(data=request.data)
         if not serializer_data.is_valid():
             return HttpResponseBadRequest(reason=serializer_data.errors)
 

diff --git a/lms/djangoapps/instructor/views/serializer.py b/lms/djangoapps/instructor/views/serializer.py
@@ -43,7 +43,7 @@ class Meta:
         fields = ['username', 'email', 'first_name', 'last_name']
 
 
-class UserAccessSerializer(serializers.Serializer):
+class AccessSerializer(serializers.Serializer):
     unique_student_identifier = serializers.CharField(
         max_length=255,
         help_text="Email or username of user to change access"