search

nus-tic4002-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Seems to not have a date validation function #110

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Description: No date validation.

Command Used: add n/John Doe d/22-02-1000 f/7 i/S2731125H p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 de/This man is a construction site manager who flew a drone over the Istana. r/He could be just curious to see what the Istana looks like. t/NeverCalled t/HighPriority

Scenario: A potential user could accidentally add in a incorrect date for the year thats unlikely. A prompt of some sort notifying the user would be nice. This is low priority though.

image.png image.png

[original: nus-tic4002-AY2021S2/pe-interim#110] [original labels: severity.Low type.FeatureFlaw]

Yiheng0410 commented 3 years ago

Team's Response

We do have a valid data validation in our app, but this is an extreme case. We agree that we could implement a minimum year and no date input later than today's date.

Duplicate status (if any):

--