nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Interview dates allow nonsensical dates #2045

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

Interview dates allow nonsensical dates like 2999, when in fact it should be impossible to schedule something past a few years.

image.png

Additionally, it also allows dates far in the past like 1000. I understand that dates recently past should be allowed due to book keeping purposes, but dates like year 1000 should not be allowed.

image.png


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

twothicc commented 2 years ago

Team's Response

We understand that a warning message should be shown if users provide a nonsensical date.

However, it is also very unlikely that users will input a date that is so ridiculously far from the current date time unless by mistake, in which case it is also easily correctable by our edit feature.

Hence, we think that this sort of input occurs very rarely and is very unlikely to affect normal operations of the application. Thus, we will reduce the severity of this bug to Low and accept it.

Duplicate status (if any):

--