feat: 158 teacher teach edit student details

backend/Pipfile

@@ -28,7 +28,7 @@ google-cloud-logging = "==1.*"
 google-auth = "==2.*"
 google-cloud-container = "==2.3.0"
 "django-anymail[amazon_ses]" = "==7.0.*"
-codeforlife = {ref = "v0.8.0", git = "https://github.com/ocadotechnology/codeforlife-package-python.git"}
+codeforlife = {ref = "v0.8.4", git = "https://github.com/ocadotechnology/codeforlife-package-python.git"}
 django = "==3.2.20"
 djangorestframework = "==3.13.1"
 django-cors-headers = "==4.1.0"

backend/portal/helpers/domain.py

@@ -5,7 +5,6 @@ def get_domain(request):
     """
     Returns the domain of the request
     """
-
     return (
         "http://localhost:3000"
         if os.environ.get("DJANGO_DEBUG") == "True"

backend/portal/serializers/__init__.py

@@ -0,0 +1,3 @@
+from .klass import ClassSerializer
+from .school import SchoolSerializer
+from .user import UserSerializer

backend/portal/serializers/klass.py

@@ -0,0 +1,6 @@
+from codeforlife.user.serializers import ClassSerializer as _ClassSerializer
+
+
+class ClassSerializer(_ClassSerializer):
+    class Meta(_ClassSerializer.Meta):
+        pass

backend/portal/serializers/school.py

@@ -0,0 +1,6 @@
+from codeforlife.user.serializers import SchoolSerializer as _SchoolSerializer
+
+
+class SchoolSerializer(_SchoolSerializer):
+    class Meta(_SchoolSerializer.Meta):
+        pass

backend/portal/serializers/user.py

@@ -0,0 +1,16 @@
+from codeforlife.user.serializers import UserSerializer as _UserSerializer
+from rest_framework import serializers
+
+
+class UserSerializer(_UserSerializer):
+    current_password = serializers.CharField(write_only=True)
+
+    class Meta(_UserSerializer.Meta):
+        extra_kwargs = {
+            **_UserSerializer.Meta.extra_kwargs,
+            "username": {"read_only": True},
+            "isActive": {"read_only": True},
+            "isStaff": {"read_only": True},
+            "dateJoined": {"read_only": True},
+            "lastLogin": {"read_only": True},
+        }

backend/portal/urls/__init__.py

@@ -1,18 +1,26 @@
 from django.urls import include, path
+from rest_framework.routers import DefaultRouter
 
+from ..views import ClassViewSet, SchoolViewSet, UserViewSet
 from .admin import urlpatterns as admin_urlpatterns
 from .cron import urlpatterns as cron_urlpatterns
 from .dotmailer import urlpatterns as dotmailer_urlpatterns
 from .email import urlpatterns as email_urlpatterns
 from .home import urlpatterns as home_urlpatterns
 from .organisation import urlpatterns as organisation_urlpatterns
-from .teacher.dashboard import urlpatterns as teach_dashboard_urlpatterns
 from .registration import urlpatterns as registration_urlpatterns
 from .student import urlpatterns as student_urlpatterns
 from .teacher import urlpatterns as teacher_urlpatterns
+from .teacher.dashboard import urlpatterns as teach_dashboard_urlpatterns
+
+router = DefaultRouter()
+router.register("classes", ClassViewSet, basename="class")
+router.register("users", UserViewSet, basename="user")
+router.register("schools", SchoolViewSet, basename="school")
 
 urlpatterns = [
     path("cron/", include(cron_urlpatterns)),
+    path("", include(router.urls)),
     *dotmailer_urlpatterns,
     *email_urlpatterns,
     *home_urlpatterns,

backend/portal/urls/teacher/dashboard.py

@@ -18,6 +18,7 @@
     create_new_class,
     get_student_request_data,
     get_students_from_access_code,
+    get_student_details,
 )
 
 from ...helpers.regexes import ACCESS_CODE_REGEX
@@ -88,4 +89,9 @@
         get_students_from_access_code,
         name="get_students_from_acccess_code",
     ),
+    re_path(
+        r"^class/student/(?P<student_id>[0-9]+)/$",
+        get_student_details,
+        name="get_student_details",
+    ),
 ]

backend/portal/urls/teacher/teach.py

@@ -5,6 +5,8 @@
     teacher_edit_class,
     teacher_delete_class,
     teacher_move_class,
+    teacher_edit_student,
+    teacher_print_reminder_cards,
 )
 
 
@@ -24,4 +26,14 @@
         teacher_move_class,
         name="teacher_move_class",
     ),
+    path(
+        "student/edit/<int:pk>",
+        teacher_edit_student,
+        name="teacher_edit_student",
+    ),
+    re_path(
+        rf"onboarding-class/(?P<access_code>{ACCESS_CODE_REGEX})/print-reminder-cards/",
+        teacher_print_reminder_cards,
+        name="teacher_print_reminder_cards",
+    ),
 ]

backend/portal/views/__init__.py

@@ -0,0 +1,3 @@
+from .klass import ClassViewSet
+from .school import SchoolViewSet
+from .user import UserViewSet

backend/portal/views/klass.py

@@ -0,0 +1,8 @@
+from codeforlife.user.views import ClassViewSet as _ClassViewSet
+
+from ..serializers import ClassSerializer
+
+
+class ClassViewSet(_ClassViewSet):
+    http_method_names = ["get", "post", "patch", "delete"]
+    serializer_class = ClassSerializer

backend/portal/views/school.py

@@ -0,0 +1,8 @@
+from codeforlife.user.views import SchoolViewSet as _SchoolViewSet
+
+from ..serializers import SchoolSerializer
+
+
+class SchoolViewSet(_SchoolViewSet):
+    http_method_names = ["get", "post", "patch"]
+    serializer_class = SchoolSerializer

backend/portal/views/teacher/dashboard.py

@@ -5,8 +5,8 @@
 from django.contrib.auth import logout
 from django.contrib.auth.decorators import login_required, user_passes_test
 from django.contrib.auth.models import User
-from django.core import serializers
-from django.db.models import F, Value
+from django.core.exceptions import ObjectDoesNotExist
+from django.db.models import F
 from django.http import (
     Http404,
     HttpResponse,
@@ -68,10 +68,12 @@
 
 @login_required(login_url=reverse_lazy("teacher_login"))
 def get_students_from_access_code(request, access_code):
-    check_teacher_authorised(request, request.user.new_teacher)
+    student_class = Class.objects.get(access_code=access_code)
+    check_teacher_authorised(request, student_class.teacher)
     students_query = Student.objects.filter(
         class_field__access_code=access_code
     )
+    # TODO: make this into a method for the student so we can reuse it
     students = [
         {
             "id": student.id,
@@ -94,6 +96,39 @@ def get_students_from_access_code(request, access_code):
     return JsonResponse({"students_per_access_code": students})
 
 
+def get_student_details(request, student_id):
+    student = get_object_or_404(Student, id=student_id)
+    try:
+        student_class = Class.objects.get(
+            access_code=student.class_field.access_code
+        )
+        check_teacher_authorised(request, student_class.teacher)
+    except (ObjectDoesNotExist, AttributeError) as error:
+        return JsonResponse({"error": str(error)})
+    # TODO: make this into a method for the student so we can reuse it
+    return JsonResponse(
+        {
+            "student": {
+                "id": student.id,
+                "class_field": getattr(student.class_field, "id", 0),
+                "new_user": {
+                    "id": getattr(student.new_user, "id", 0),
+                    "first_name": getattr(student.new_user, "first_name", ""),
+                    "last_name": getattr(student.new_user, "last_name", ""),
+                },
+                "pending_class_request": getattr(
+                    student.pending_class_request, "id", 0
+                ),
+                "blocked_time": student.blocked_time.strftime(
+                    "%Y-%m-%d %H:%M:%S"
+                )
+                if student.blocked_time
+                else "",
+            }
+        }
+    )
+
+
 def _get_update_account_rate(group, request):
     """
     Custom rate which checks in a POST request is performed on the update