yixiann / pe

0 stars 0 forks source link

Incorrect error message for correct date format. #9

Open yixiann opened 1 year ago

yixiann commented 1 year ago

image.png

The error message specified is not accurate in responding to a date with a correct format.

nus-se-bot commented 1 year ago

Team's Response

The error message is in fact correct, though it lacks specificity.

The 'Original' Bug

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

Error message for dates can be more specific

Error message is correct but not specific enough

The date in the first example below is a format error (reason: MM should be in 1..12)

The date in the second example below is an invalid input (reason: February doesn't have 30 days)

image.png

image.png


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

Their Response to the 'Original' Bug

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

The error message given informs the user that the given date must both be valid and in the correct format. Although it is not super specific, differentiating between the 2 types of incorrect inputs is not essential to the use of the app. More specific error messages can be implemented in later iterations.

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: [replace this with your explanation]


:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]