feat: Add PATCH endpoint to accept mentor registration by lauracabtay · Pull Request #516 · Women-Coding-Community/wcc-backend

lauracabtay · 2026-02-11T21:12:05Z

Description

This PR introduces a new workflow for mentor registration and approval, ensuring that mentor profiles are initially set to a PENDING status and can only be activated through a dedicated endpoint (PATCH /api/platform/v1/mentors/{mentor_id}/accept. It also adds exception handling for mentor status conflicts and updates API documentation and request validation. The changes are grouped below by theme:

Mentor Registration & Approval Workflow:

Added CreateMentorRequest DTO for mentor registration, enforcing validation.
Updated MentorDto and its toMentor() method to enforce profile status as PENDING on creation, preventing clients from setting it directly.
Modified the mentor creation endpoint in MentorshipController to use CreateMentorRequest.
Added a new PATCH endpoint /mentors/{mentorId}/accept in MentorshipController to activate a mentor's profile, changing their status to ACTIVE.
Introduced MentorStatusException and added handling in GlobalExceptionHandler to return HTTP 409 (Conflict) for mentor status-related errors (i.e. if status is already active).
Added updateProfileStatus method to MentorRepository for updating mentor profile status.
Updated Postman collection and scripts to reflect new mentor registration and approval workflow, including removal of profileStatus from request bodies and addition of the new approval endpoint.
Added new template type MENTOR_APPROVAL to TemplateType enum for mentor registration approval notifications.
Introduced a NotificationService to handle the sending of notifications (at the moment emails) related to mentor and mentee events. It centralises notification logic, making it easier to manage different types of notifications (e.g., mentor approval, mentee onboarding) in one place (note: Ability to send email is pending Infrastructure: Setup Gmail SMTP Authentication #448 and Setup: Generate SMTP Authentication for dev and prod #449)

Related Issue

Parent: #488
Closes #509

Change Type

Screenshots

CreateMentorRequest schema:

ProfileStatus is not posted in the request body:

Newly created mentor shows as pending:

Mentor set to active:

Mentor is already active:

Pull request checklist

Please check if your PR fulfills the following requirements:

I checked and followed the contributor guide
I have tested my changes locally.

…istration # Conflicts: # src/main/java/com/wcc/platform/domain/platform/mentorship/MentorDto.java # src/test/java/com/wcc/platform/domain/platform/mentorship/MentorDtoTest.java

src/main/java/com/wcc/platform/controller/MentorshipController.java

src/main/java/com/wcc/platform/service/MentorshipService.java

src/main/resources/email-templates/mentor_registration_approval.yml

src/main/java/com/wcc/platform/service/NotificationService.java

src/main/resources/email-templates/mentor_registration_approval.yml

# Conflicts: # src/test/java/com/wcc/platform/service/MentorshipServiceTest.java

src/main/java/com/wcc/platform/domain/platform/mentorship/MentorDto.java

src/main/java/com/wcc/platform/service/MentorshipNotificationService.java

src/main/java/com/wcc/platform/configuration/NotificationTemplateConfig.java

…tion

src/main/java/com/wcc/platform/domain/template/TemplateType.java

src/main/resources/email-templates/mentorship_profile_approved_mentor.yml

lauracabtay added 11 commits February 10, 2026 08:54

Create accept mentor endpoint

46a662a

Create NotificationService and send approval email to Mentor

9dc2e80

Add tests and tidy up

a83f850

Tidy up

f9b9ebe

Add CreateMentorRequest for request body

a3d2df5

Merge remote-tracking branch 'origin/main' into feat-patch-mentor-reg…

d1a5535

…istration # Conflicts: # src/main/java/com/wcc/platform/domain/platform/mentorship/MentorDto.java # src/test/java/com/wcc/platform/domain/platform/mentorship/MentorDtoTest.java

Tidy up CreateMentorRequest

cabcd09

Update openapi.yaml / postman.json

6e94a5e

Remove availability from CreateMentorRequest

30bf63e

Fix required fields on swagger schema

190e666

Fix required fields on swagger schema

4559628

lauracabtay requested a review from a team as a code owner February 11, 2026 21:12

lauracabtay requested review from dricazenck, goelsonali, gpaOliveira, nora-weisser, nverbic, silkenodwell and wilsonadenuga February 11, 2026 21:12

lauracabtay added 2 commits February 11, 2026 21:20

Remove unnecessary getter and fix failing test

a448c22

Suppress PMD warnings

799443c