search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Illegal time field allowed. #2591

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

UG says timeshould be entered in format HHMM. However, illegal foramt as shown above is also allowed

command: add_event Date Night place/MBS time/000000300 m/2

1.jpg

[original: nus-cs2103-AY1920S2/pe-interim#2589]

l2ejected commented 4 years ago

Team's Response

The UG states that time should be entered in the HHMM format. We accept that we did not clearly state the upper limits of the maximum number of characters for the input for the time for an event. But functionally, the app still records the time correctly as 3 hours. It is a failing on our part to not explicitly state the maximum number of hours that can be added and how the app handles input time with 0's at the front

Duplicate status (if any):

--