search

mmcknett / loggr

Time logging system
https://loggr-a3f89.web.app/
0 stars 0 forks source link

End time can be before start time #31

Open amythetester opened 1 year ago

amythetester commented 1 year ago

User is able to enter an earlier end time than the start time

mmcknett commented 1 year ago

Good find. End time before start time should show a validation error and inhibit the submit button.

I don't think there would be any good reason to preserve this behavior. If for some reason users need a "negative time span" feature, it should probably be thought about independently with a different storage mechanism. Any other strange date-related corner cases, such as logging at 1:30am on "fall back" day or logging an event that crossed time-zones, would be better handled by handling time zones explicitly.