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

Input validation #104

Open
AlexHalbesleben opened this issue Aug 19, 2022 · 0 comments · May be fixed by #157
Open

Input validation #104

AlexHalbesleben opened this issue Aug 19, 2022 · 0 comments · May be fixed by #157
Labels
bug Something isn't working ui Issues relating to or with resolutions that will impact the user interface

Comments

@AlexHalbesleben
Copy link
Owner

The site suffers from a lack of input validation pretty much everywhere. I believe there is a single check in the settings modal, but there should be more checks to make sure that inputs make sense. Some examples:

  • Tasks must have a name (no empty names or names with only whitespeace)
  • Tasks cannot be due before the current date
  • Tasks cannot have a negative duration, effort, priority (eventually), or number of chunks
  • Chunks cannot be manually assigned after their parent task's due date or before the current date
  • Anywhere where a range of dates/times can be edited, the end date/time must be after the start date/time
@AlexHalbesleben AlexHalbesleben added bug Something isn't working medium-priority ui Issues relating to or with resolutions that will impact the user interface labels Aug 19, 2022
@AlexHalbesleben AlexHalbesleben added this to the v4.0 milestone Aug 19, 2022
@AlexHalbesleben AlexHalbesleben self-assigned this Aug 19, 2022
@AlexHalbesleben AlexHalbesleben removed this from the v4.0 milestone Aug 22, 2022
@AlexHalbesleben AlexHalbesleben removed their assignment Aug 22, 2022
@AlexHalbesleben AlexHalbesleben linked a pull request Aug 31, 2022 that will close this issue
@AlexHalbesleben AlexHalbesleben linked a pull request Aug 31, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Issues relating to or with resolutions that will impact the user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant