search

neojxuan / pe

0 stars 0 forks source link

Incorrect error message #8

Open neojxuan opened 2 years ago

neojxuan commented 2 years ago

000 is a valid three-digit integer so the command format is valid. However, the error message given is not that the list does not contain the id.

image.png

nus-pe-bot commented 2 years ago

Team's Response

Thank you for the response :), this is a duplicate bug

The 'Original' Bug

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

Different error messages for invalid patient index

image.png

image.png

The invalid index 'P001' should be identified as an invalid index to identify the correct error to users, as the invalid command format error message does not specify the issue with the index.

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

Their Response to the 'Original' Bug

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

Thank you for the response :) This does not cause any issue with the usage and the error message is a purely cosmetic issue.

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]