xz0127 / pe

0 stars 0 forks source link

Inaccurate error message for delete command #3

Open xz0127 opened 3 years ago

xz0127 commented 3 years ago

When using "delete" command with a valid ID, it would to better to show the animal does not exist instead of the ID is invalid for error message. Because user may think it is due to their wrong input format. Screenshot 2020-11-13 at 16.29.47.png

nus-se-bot commented 3 years ago

Team's Response

Cosmetic flaw, does not affect functionality. We discussed this before implementation and agreed on the same error message. However, we agree that your suggestion is more specific and clear.

Items for the Tester to Verify

:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]