Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scheduling: Multiple event scheduling nitpicks #1077

Open
gregorydlogan opened this issue Jan 16, 2025 · 0 comments
Open

Scheduling: Multiple event scheduling nitpicks #1077

gregorydlogan opened this issue Jan 16, 2025 · 0 comments
Labels
priority:low type:bug Something isn't working

Comments

@gregorydlogan
Copy link
Member

gregorydlogan commented Jan 16, 2025

Multiple minor issues:

  • Multiple event scheduling allows scheduling events in the past. Single event scheduling (correctly) refuses to let me do this.
  • Modal does not check if the date range, and day-of-the-week settings make sense.
    • I would expect settings like Monday and Tuesday, but restricting the date range to 25-01-16 -> 25-01-17 to throw an error in the UI rather than after the final submit to the backend
    • Assuming at least one day and date combination match things work as expected.
  • Changing the hour (or duration) such that the end time is tomorrow does not display the date next to the end time the way that the single-event scheduling does. Obviously we can't put a date here, but something like + 1 day or something next to it would clarify.
    • Not supporting this at all would also be an option here. This functionality raises a bunch of fun edge cases.

Noticed in testing #1069

@gregorydlogan gregorydlogan added priority:low type:bug Something isn't working labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant