Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Conversation

@venxer
Copy link
Contributor

@venxer venxer commented Nov 20, 2024

What?

  • Utilized API for course information
  • Edited filtering menu

Why?

  • To use proper data instead of test data
  • Make the menu more user-friendly

How?

  • Connected with frontend via API calls
  • Edited Menu CSS

Testing?

  • Not broken

Anything Else?

  • N/A

liug88 and others added 11 commits November 15, 2024 16:32
First API Testing!
Added AXIOS to package
Display course ID and Code, course name.
Display credits as range if range exist.
- Fixed fetching from API
- Search on enter
Background darker for clarity
Course Attributes and Semesters offered are now underneath the course description
- Shows select attribute menu on top
- Edited width of menu
Updated Blurb Margins
Filters are saved saved when it is opened and closed.
Fixed merged conflict with toolbox
@WChen04 WChen04 self-requested a review November 22, 2024 19:55
Copy link
Contributor

@WChen04 WChen04 left a comment

Choose a reason for hiding this comment

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

Looks good!

@WChen04 WChen04 merged commit 0985d39 into main-preview Nov 22, 2024
1 check passed
@LawrenceMiao LawrenceMiao deleted the course-search-attributes branch January 28, 2025 21:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants