search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

inproper exception handling in update event #1621

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

image.png When updating an event, if the time format is entered incorrectly, the error message shows that the time format is incorrect but it goes back to main menu directly. The user can't retype the date / time, but instead the user has to restart from the first command again.   image.png

Moreover, the date time format is not specified in the user guide whether 2400 is allowed. Because 0000 is allowed but 2400 is not allowed.

[original: nus-cs2113-AY2324S2/pe-interim#1429] [original labels: severity.Low type.FeatureFlaw]

kyhjonathan commented 4 months ago

Team's Response

It is common knowledge that the time format is from 0000-2359. Hence it is expected for the program to reject 2400.

Below link shows evidence that 2400 is only used in military context, which is not our target audience specified.

https://www.google.com/search?client=firefox-b-d&q=is+0000+or+2400+more+commonly+used#ip=1

Duplicate status (if any):

--