opportunity-hack / evs

Equestrian Volunteer Scheduler

https://www.ohack.dev/nonprofit/89076bcc077811edbfdeb29c4ac23549

MIT License

1 stars 2 forks source link

Past events persist when filtering to show only events that need volunteers #70

Closed komal-saini closed 11 months ago

komal-saini commented 11 months ago

Overview The calendar feature includes a toggle to 'Show only events that need more volunteers.' However, when this filter is activated, it does not hide past events as expected. This issue leads to a cluttered view and confusion for users trying to find events to volunteer for.

Suggestion Implement a check within the filter function to ensure that only future events are displayed when the toggle is activated.

parkerdavis1 commented 11 months ago

Good call! Modified eventsThatNeedHelp filter to exclude past events: https://github.com/opportunity-hack/evs/commit/347faad84c162b5beafc4a7bb143bf8ccc512de3

gregv commented 11 months ago

Validated in prod - ty @parkerdavis1 and @komal-saini !! 🎉🎉🎉🎉🎉

Before

After

Created event that occurs in 1 hour and tested to ensure this works for same-day