nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Adding appt at 00:00 fails even though it is valid based on UG #1432

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Reproduce:

  1. appt -a p/1 d/2 s/13/11/2021 00:00

This command does not work even though the values 0 for both minute and hour is valid as it is mentioned that minutes must be between 0-59 and hours must be between 0-23. Perhaps this should be conveyed more clearly to the user or parsed to the next day.

image.png


[original: nus-cs2103-AY2122S1/pe-interim#1486] [original labels: type.FunctionalityBug severity.High]

ChuangZheQuan commented 3 years ago

Team's Response

Screenshot 2021-11-14 at 9.45.38 AM.png

Screenshot 2021-11-14 at 9.46.19 AM.png

This bug report is invalid.

We are able to add the appointment. I think you added 2 spaces between the date and time, resulting in the format error.

Duplicate status (if any):

--