search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Inaccurate error message for remind #1534

Open nus-se-script opened 9 months ago

nus-se-script commented 9 months ago

image.png Rather than The value is not a number, it should probably be The value is not a positive integer

[original: nus-cs2103-AY2324S1/pe-interim#1241] [original labels: type.DocumentationBug severity.VeryLow]

brendanneojw commented 9 months ago

Team's Response

I accept this as a valid bug as I did not consider users keying in numbers with decimal places. A number with decimal places is indeed still a number just that I should have displayed the error message as what you suggested. Thank you for pointing it out.

Duplicate status (if any):

--