Skip to content

Gc 47 mentorship matching#255

Merged
AidanKelly50 merged 13 commits intomainfrom
gc-47-mentorship-matching
Mar 30, 2026
Merged

Gc 47 mentorship matching#255
AidanKelly50 merged 13 commits intomainfrom
gc-47-mentorship-matching

Conversation

@filic2005
Copy link
Copy Markdown
Contributor

Description

Implements mentorship matching automation and several mentorship dashboard improvements. When a mentee signs up or a mentor is activated, the matching algorithm now runs automatically (with a guard for no available mentors). Mentors and mentees can opt in/out of new matches. Mentors receive push notifications on new requests. Includes an admin stats dashboard and expanded mentor review modal.

Ticket URL

GC-47, GC-49, GC-50, GC-51, GC-53, GC-63, GC-64

Type of Change

  • Feature
  • Bug fix

Scope of Change

  • Frontend
  • Backend
  • Data

Testing

  • Registered mentor and mentee accounts locally
  • Verified algorithm runs automatically when mentor is activated with waiting mentees
  • Verified opt-in/out toggle updates DB and filters recommendations
  • Verified admin stats page loads correct counts at /mentorship/admin
  • Verified push notification sent to mentor on mentorship request
  • Verified mentor review modal shows expanded profile fields
  • Verified dynamic dashboard headers update based on match state

@linear
Copy link
Copy Markdown

linear bot commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

@AidanKelly50 AidanKelly50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverts changes I made earlier in the semester regarding meeting location preferences.

@AidanKelly50 AidanKelly50 merged commit 806b421 into main Mar 30, 2026
2 checks passed
@AidanKelly50 AidanKelly50 deleted the gc-47-mentorship-matching branch March 30, 2026 22:41
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