Merge pull request #16625 from yafred/teacher-not-updated-on-move

update teacher info after transferring student to another class
lichess-org · Dec 19, 2024 · 7a032e1 · 7a032e1
2 parents df31833 + 2115237
commit 7a032e1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/modules/clas/src/main/ClasApi.scala b/modules/clas/src/main/ClasApi.scala
@@ -260,7 +260,11 @@ final class ClasApi(
 
     def move(s: Student.WithUser, toClas: Clas)(using teacher: Me): Fu[Option[Student]] = for
       _ <- closeAccount(s)
-      stu = s.student.copy(id = Student.makeId(s.user.id, toClas.id), clasId = toClas.id)
+      stu = s.student.copy(
+        id = Student.makeId(s.user.id, toClas.id),
+        clasId = toClas.id,
+        created = Clas.Recorded(by = teacher.userId, at = nowInstant)
+      )
       moved <- colls.student.insert
         .one(stu)
         .inject(stu.some)