search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

App does not account for small typo in format #1549

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months ago

image.png When time is inputted as "23.40" instead of "23:40", App rejects the input

[original: nus-cs2103-AY2324S2/pe-interim#1959] [original labels: type.FunctionalityBug severity.Low]

alfaloo commented 5 months ago

Team's Response

Both the error message and the user guide explicitly states the input should be "yyyy-MM-dd HH:mm". The MediCLI product is designed to be used as specified in the user guide and internal error messages. Furthermore, our addappt command is already lenient of small typos. For example, for the date-time field, extra white space between the date and the time will be ignored (because we understand for fast typing clerks, extra space is an easy mistake). However, the mistake being reported is not a very likely typo by a skilled typer (the intended audience of the 2103 project). Thus we don't think this is a valid bug.

Duplicate status (if any):

--