Skip to content

Feat/adhoc application flow#716

Merged
goelsonali merged 2 commits into
Women-Coding-Community:mainfrom
Sowmiya07:feat/adhoc-application-flow
Jun 25, 2026
Merged

Feat/adhoc application flow#716
goelsonali merged 2 commits into
Women-Coding-Community:mainfrom
Sowmiya07:feat/adhoc-application-flow

Conversation

@Sowmiya07

Copy link
Copy Markdown
Contributor

Description

  • AD_HOC mentee applications are created directly in MENTOR_REVIEWING status, bypassing the admin approval step
    • acceptApplication delegates to MentorshipMatchingService.confirmMatch() for AD_HOC cycles, moving directly to MATCHED
    • confirmMatch validates expected status based on cycle type: MENTOR_REVIEWING for AD_HOC, MENTOR_ACCEPTED for LONG_TERM
    • Mentor is notified via sendNewMenteesNotification at registration time for AD_HOC (mirrors approveApplication behaviour for LONG_TERM)
    • TODO added on MentorshipMatchingService injection in MenteeWorkflowService to flag cyclic-dependency risk for future
      refactor

Related Issue

#712

Change Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation
  • Test
  • Other

Screenshots

image image image image image image

Pull request checklist

Please check if your PR fulfills the following requirements:

@goelsonali goelsonali merged commit a62fd63 into Women-Coding-Community:main Jun 25, 2026
5 of 6 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants