nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Invalid year accepted #134

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

image.png

Expected: Error message is shown as years like 9999 or 0001 are impossible to reach

Actual: Input is accepted

Suggestion: Might be better to set a restriction on the years


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

Jaynon commented 2 weeks ago

Team's Response

image.png

The year 0001 and 9999 exists and are considered extreme inputs, there is no harm accepting such years. it is also subjective to decide which years are the hard limit.

Duplicate status (if any):

--