nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Training time input not checked #2599

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

The time keyed in is not a valid time, yet it was accepted as an input

Screenshot 2021-11-12 at 4.45.57 PM.png


[original: nus-cs2113-AY2122S1/pe-interim#2553] [original labels: severity.Medium type.FunctionalityBug]

ianwangeeen commented 2 years ago

Team's Response

The '/a' portion is meant to be keyed in as a string. Thus, dates such as 'Friday PM' or 'Friday 1200' or '1200' are all accepted and meant to be allowed for the user to format himself.

This is also justified in the UG

image.png

Duplicate status (if any):

--