Skip to content

Course details page#408

Open
lyangji1011 wants to merge 15 commits intoemily/395-course-details-pagefrom
lauren/course-details-page
Open

Course details page#408
lyangji1011 wants to merge 15 commits intoemily/395-course-details-pagefrom
lauren/course-details-page

Conversation

@lyangji1011
Copy link
Contributor

@lyangji1011 lyangji1011 commented Nov 11, 2025

Summary

Resolves #397

Checklist

  • Add a new field in the context that determines if we are on the calendar page, the course details page (general), or the course details page for a selected class (it should keep a record of that class).
  • Where the schedule is displayed right now, add a new View Mode button above with the Calendar and Course Details option. You can use the TabBar component for this. This will alter the context field you added.
  • Create the empty course details page that displays if there are no courses in the left side panel.
  • Create the CourseDetailsContainer that takes in no props and displays all the different CourseInfoCards for all classes in the schedule context.
  • Make the SectionDetailsContainer that uses whatever selected class in the context to display the following using a Breadcrumb and the ClassInfo.
  • Add the info icon to the class in the left panel
  • Create all the flows and all the functionalities/tab switching by updating the new context field on whatever actions.

How to Test

DanielChee and others added 10 commits November 1, 2025 18:35
### Summary

Resolves #396 

adds location in a new line in Map tab sidebar

### How to Test
add a class and section with a location
check location appears in Map tab

---------

Co-authored-by: afazio1 <alexa.fazio04@gmail.com>
@lyangji1011 lyangji1011 linked an issue Nov 12, 2025 that may be closed by this pull request
7 tasks
@lyangji1011 lyangji1011 marked this pull request as ready for review November 14, 2025 22:49
@uma-anand uma-anand changed the base branch from bog-f25-sprint5 to emily/395-course-details-page November 17, 2025 23:22
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.

[BoG F25] Course Details Page: And that's a wrap!

3 participants