Skip to content

feat(uc-17): implement delete student - #70

Merged
visiort merged 1 commit into
mainfrom
feature/uc-17
Apr 26, 2026
Merged

visiort merged 1 commit into
mainfrom
feature/uc-17

Conversation

@visiort

@visiort visiort commented Apr 26, 2026

Copy link
Copy Markdown
Collaborator

Backend:

  • StudentService with findAllStudents, findStudentById, deleteStudent
  • UserController with GET/DELETE /users/students endpoints (admin-only)
  • StudentNotFoundException, StudentSummary, StudentDetail DTOs
  • UserRepository: findAllStudents and findStudentById queries
  • TeamRepository: findAllByStudentId query
  • UserExceptionHandler: handle StudentNotFoundException with 404
  • UserControllerIntegrationTest covering all endpoints and auth cases

Physical delete cascades to WARs, peer evaluations, and team memberships via ON DELETE CASCADE constraints already in the DB schema.

Frontend:

  • Students list page with deletion notification banner
  • StudentDetail page with two-step delete confirmation dialog
  • studentService, studentTypes, studentNotifications store
  • Routes: /students and /students/:id (admin-only)

Backend:
- StudentService with findAllStudents, findStudentById, deleteStudent
- UserController with GET/DELETE /users/students endpoints (admin-only)
- StudentNotFoundException, StudentSummary, StudentDetail DTOs
- UserRepository: findAllStudents and findStudentById queries
- TeamRepository: findAllByStudentId query
- UserExceptionHandler: handle StudentNotFoundException with 404
- UserControllerIntegrationTest covering all endpoints and auth cases

Physical delete cascades to WARs, peer evaluations, and team memberships
via ON DELETE CASCADE constraints already in the DB schema.

Frontend:
- Students list page with deletion notification banner
- StudentDetail page with two-step delete confirmation dialog
- studentService, studentTypes, studentNotifications store
- Routes: /students and /students/:id (admin-only)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@visiort
visiort merged commit 7ffdf10 into main Apr 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant