Skip to content

Conversation

@vicksey
Copy link
Contributor

@vicksey vicksey commented Dec 4, 2025

Summary

Uses data from PeterPortal to filter out classes that users have taken before the current quarter in their four-year plan.
Re-pulled the logic from old instance of #1274 (og pr got messy and blocked last yr with auth and new ui design).

Test Plan

  • Log into a users' account and test filter logic to ensure correct classes are being filtered
  • Toggle on/off the filter feature (UI)
  • Search for a course the user's already taken in their selected roadmap shouldn't show up in fuzzy search
  • Search for a course the user's already taken in their selected roadmap shouldn't show up course render
  • Search for a future (planned) course on a user's roadmap - it should still show up
  • Filtering logic is just for selected roadmap
  • Shouldn't filter if toggled off
  • Make sure search is still good

Issues

TODO:

  • FuzzySearch filtering

Closes #1028 #1274

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.

RFC: Filter out classes already taken

2 participants