Skip to content

update generation algo to take into account coreqs#386

Open
ItsEricSun wants to merge 5 commits intomainfrom
fix-scheduler-coreqs
Open

update generation algo to take into account coreqs#386
ItsEricSun wants to merge 5 commits intomainfrom
fix-scheduler-coreqs

Conversation

@ItsEricSun
Copy link
Copy Markdown
Contributor

Pull Request

  • Updated schedule generation to make sure returned schedules contain all passed in coreqs
  • Updated add course modal to count coreqs together as 1 course instead of however distinct courses they are
  • Updated add course modal to use the same term as the saved plan rather than just defaulting to the most recent term
  • Fix auto plan naming to be an incrementing integer

Closes https://sandboxneu.slack.com/archives/C0993L6GW10/p1774828388053049

Type of Change

Please tick the boxes that best match your changes.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change requires a PCP (ie changes in deps, database, infrastructure, or package exports)

Testing

Please describe how you tested this PR (both manually and with tests) Provide instructions so we can reproduce.

Checklist

  • I have performed a self-review of my own code
  • I have commented my code where needed
  • I have made corresponding changes to the documentation
  • I have run a build for the entire monorepo
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • All commits are atomic and my branch is cleaned (no WIP commits)

@ItsEricSun ItsEricSun self-assigned this Mar 30, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
searchneu Ready Ready Preview, Comment Apr 6, 2026 9:40pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
searchneu-docs Skipped Skipped Apr 6, 2026 9:40pm

Request Review

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.

2 participants