Education tab Mentor section#267
Merged
MBreathe merged 10 commits intoHackYourFuture:mainfrom Jan 26, 2026
Merged
Conversation
… added new mentors section to education tab, also tested for editing
MBreathe
requested review from
Aya-Alabrash,
sima-milli and
stasel
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request introduces support for tracking mentor information (technical, HR, and English mentors) in trainee education records across all layers of the application stack. The changes are implemented consistently from the database schema through the API to the frontend UI.
Changes:
- Added three new optional mentor fields (
techMentor,hrMentor,engMentor) to theTraineeEducationInfointerface across both backend and frontend models - Updated the Mongoose schema, API documentation, and React UI component to support these new fields
- Upgraded multiple server and client dependencies to newer versions for security and compatibility improvements
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| server/src/models/Trainee.ts | Added three optional mentor field types to the TraineeEducationInfo interface |
| server/src/schemas/TraineeSchema.ts | Updated Mongoose schema to persist mentor fields with proper defaults |
| server/api.yaml | Extended OpenAPI specification to document the new mentor fields |
| client/src/models/Trainee.ts | Added mentor fields to frontend TypeScript interface matching backend |
| client/src/components/education/EducationInfo.tsx | Implemented UI form fields for displaying and editing mentor information |
| server/package-lock.json | Updated multiple dependencies including axios, body-parser, debug, and AWS SDK packages |
| client/package-lock.json | Updated multiple dependencies including esbuild, vite, react-router, and related packages |
Files not reviewed (2)
- client/package-lock.json: Language not supported
- server/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
MBreathe
force-pushed
the
feature/EducationT_mentor-info
branch
from
8af700b to
606505a
Compare
HackYourFutures
temporarily deployed
to
dojo-feature-educationt-b5ltwe
stasel
reviewed
Jan 20, 2026
Member
|
Looks very nice. Can you let it fill the width of the screen a bit more? Similar to the contact or personal tabs. Something like until the red line in this screenshot 
|
… be wider and resemble contact tab fields
HackYourFutures
temporarily deployed
to
dojo-feature-educationt-b5ltwe
HackYourFutures
temporarily deployed
to
dojo-feature-educationt-b5ltwe
HackYourFutures
temporarily deployed
to
dojo-feature-educationt-b5ltwe
stasel
approved these changes
Jan 25, 2026
HackYourFutures
temporarily deployed
to
dojo-feature-educationt-b5ltwe
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces support for tracking mentor information (technical, HR, and English mentors) in trainee education records across the application. It updates the frontend, backend, and API schema to include these new fields, and also bumps several server dependencies to newer versions.
Mentor information feature:
Added
techMentor,hrMentor, andengMentorfields to theTraineeEducationInfointerface in both frontend (client/src/models/Trainee.ts) and backend (server/src/models/Trainee.ts).Updated the trainee education info schema to persist the new mentor fields in the database (
server/src/schemas/TraineeSchema.ts).Extended the OpenAPI schema to document the new mentor fields in the API (
server/api.yaml).Updated the
EducationInfoReact component to display and allow editing of the mentor fields in the UI (client/src/components/education/EducationInfo.tsx).Dependency updates:
Upgraded several server dependencies in
server/package-lock.json, includingaxios,body-parser,debug,iconv-lite,js-yaml,jws, and multiple@smithy/*packages, improving security and compatibility.