seas-computing / course-planner

SEAS Course Planning Application. JSDoc Docs at: https://seas-computing.github.io/course-planner/
0 stars 1 forks source link

"Same As" Courses cannot be unpaired #650

Closed jonseitz closed 1 year ago

jonseitz commented 1 year ago

It seems that a "sameAs" course cannot be unpaired after being set. When Sarah Colgan tried to remove ES 237 from EPS 237, she was able to save the course but afterwards the two are still linked in the database. This might be an over-eager validation step that's interpreting a blank "SameAs" field as undefined, and thus skipping it when updating a course.

Sarah's email:

I’m hoping you can help with a specific case: I’d like to unpair ES 237 from EPS 237. I tried to remove ES 237 from EPS 237, but it doesn’t stick. These courses were co-listed through spring 2019, but we no longer allow co-listing grad courses across EPS and ES, so I’d like to unpair and then retire ES 237. Going forward, it will only be offered as an EPS course (which will be offered in Spring 2024).