senior-knights / course-schedulizer

📝 Create semester schedules without stress
https://senior-knights.github.io/course-schedulizer/
MIT License
10 stars 4 forks source link

Conflict message issue when removing constraints #233

Open das43 opened 2 years ago

das43 commented 2 years ago

We found that when uploading a constraint file, the proper conflict message in the section popup appears, but then when we remove the constraints by uploading a blank file, the conflicts get removed in the conflict tab but the conflict message now claims there is an instructor conflict with itself.

We aren't exactly sure why this is happening and don't have time to fix it right now. The messages in the popup are fetched from the conflictRows part of the schedule object, so maybe it's not removed there, or possibly the function that takes our simplified constraint file and expands it so all combinations of classes work.

Opening the now falsely conflicting class and changing its duration (and then changing it back so it's correct) seems to fix the issue.