search

CShuwen / pe

0 stars 0 forks source link

Accept invalid dates #5

Open CShuwen opened 1 year ago

CShuwen commented 1 year ago

Screenshot 2023-04-14 145949.png

Screenshot 2023-04-14 150023.png

The date 2023-02-29 should be an invalid date, however, no error message is shown and the date is updated to 2023-02-28, which changes what the user intend to input without notifying the user.

nus-se-script commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Dates in February being unclear regarding having errors or not

Screenshot 2023-04-14 at 2.23.20 PM.png

writing edge case dates like February 31 will not cause an error

Screenshot 2023-04-14 at 2.24.41 PM.png

but it will silently put in the most latest valid date in february, not any breaking bugs but might cause confusion

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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This is confusing for the users. The app is not showing what the users are intending it to see. Minimally your team should have mentioned this in your user guide but you didn't.