search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

List command shows 29 Feb 0001, but summarise errors out #2531

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month ago

image.png

I would say the command itself is correct (there is no 29 feb 0001) but because of the fact that the date got accepted, I am treating that the date 29 Feb 0001 exists here

This is related to the 29 Feb 0000 being accepted and changed to 29 Feb 0001 issue. Fixing the date issue in my other issue opened will also resolve this.

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

yeozhishen commented 3 weeks ago

Team's Response

Fixing the bug associated with #2387 will resolve the bug with this issue. Both issues are caused by the issue of not filtering out extreme dates, which can be resolve by adding more checks to the application's dateParser.

Duplicate status (if any):

Duplicate of #2387