search

blaukc / pe

0 stars 0 forks source link

Date parsing when time is 24:00 #10

Open blaukc opened 2 months ago

blaukc commented 2 months ago

event n/test at/2023-02-27 24:00 h/1 When typing in this command, i get the event set to the next day. However, the expected behaviour would be to show an error message that 24:00 is not a valid time.

image.png

nus-pe-script commented 2 months ago

Team's Response

Thank you for your response. However, according to the ISO, 24:00 of one day is the same time as 00:00 of the following day. Several other sources back this claim. See here

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]