nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Date beyond range #5423

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 months ago

image.png

App takes in dates beyond a realistic range.


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

iynixil commented 5 months ago

Team's Response

Thanks for raising this. This is intended, and was mentioned in the UG under Adding a meeting to a person: meeting-add section.

image.png

Our definition at any point of time refers to a specific part of the timeframe that has existed, currently existing or will exist in the future. Another reason why we do not want to limit the timeframe as the definition of realistic range is too broad and general. There is no concrete and clear-cut number that will make everyone agree that it is realistic. Hence, our team agrees to satisfy everyone and meet the requirement of our software we will allow any valid date that has a possibility to exist at some point of time. Nonetheless some might find it as garbage values , but our software delivers correctly as it is told to. (Garbage in, garbage out)

Duplicate status (if any):

--