search

rpiambulance / website

RPI Ambulance's website—a work in progress
http://rpiambulance.com/
3 stars 13 forks source link

Fixes issue with 2 week rotation #110

Closed jcub closed 1 year ago

jcub commented 4 years ago

where if you sign up for the future week you can still sign up for the current week.

ddbruce commented 4 years ago

@MasterOdin @jcub any further discussion on the above?

jcub commented 4 years ago

Like I said, if it is going to be changed we should change it across the board (everywhere sizeof is used vs count). This should be done separate from this PR. The current version works however there was discussion on if the "issue" is an issue or a feature and also there was further discussion on if the fix needs to be implemented.

ddbruce commented 1 year ago

I don't believe this is going to work properly. New PR incoming soon