Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 9, 2025

The dashboard only supported displaying a single mentor. Extended to support two mentors with minimal changes to data structure and rendering logic.

Changes

  • data/mentor.json: Changed mentor object to mentors array containing two mentor entries
  • components/mentor.js:
    • Maintains backwards compatibility with single-mentor format via fallback: config.mentors || (config.mentor ? [config.mentor] : [])
    • Renders multiple mentor cards using .map() in both view and edit modes
    • Extracted getDefaultMentorName() helper to reduce duplication
    • Added proper bounds checking for form input handling
  • styles.css: Added grid layout to #mentor-details for side-by-side display

Structure

// Old format (still supported)
{ "mentor": { "name": "...", ... }, "feedback": [...] }

// New format
{ "mentors": [
    { "name": "Mentor 1", ... },
    { "name": "Mentor 2", ... }
  ], 
  "feedback": [...] 
}

Screenshots

View Mode
View Mode

Edit Mode
Edit Mode

Original prompt

This section details on the original issue you should resolve

<issue_title>Add a space for two mentors</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add a space for two mentors Add support for two mentors in dashboard Dec 9, 2025
Copilot AI requested a review from DonnieBLT December 9, 2025 06:56
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.

Add a space for two mentors

2 participants