senior-knights / course-schedulizer

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

Feature/standard times issue 300 #301

Closed rpruim closed 1 year ago

rpruim commented 1 year ago

As per #300, I've made the standard time check a bit looser. This will allow people to schedule classes that meet only one time from a meeting pattern without being flagged.

I've implemented this be expanding the list of legal meeting patterns when it is created to include the one-day versions of teach meeting pattern. Everything else is just as before.

I've added several additional test cases to confirm the new behavior is working as desired.

Side note: See #295. This would be cleaner if I didn't need to convert from TH to R and back again.

rpruim commented 1 year ago

I'm confused about this PR. It appears the changes it is intended to introduce are already in production. Furthermore, there is a conflict with a subsequent update to the standard meeting patterns. I've twice tried to resolve those on github, but they don't seem to take.

I'm going to close this since I don't think it is actually needed.

charkour commented 1 year ago

It appears the changes it is intended to introduce are already in production.

Did you push to production on accident at one point? Or were they included with a different feature branch? It seems the latter is most likely (I've done that before).

rpruim commented 1 year ago

I don't think I can push to production. (I tried it once to see if that's what happened and my push was blocked.)

I'm not sure just what happened, but if you are interested, this is roughly how things went down (IIRC):

charkour commented 1 year ago

Ah yeah, sounds like a complicated git workflow. I wish GitHub had the option to disallow fixing conflicts online. Often times, it doesn't work quite right.