ThomasHoooo / pe

0 stars 0 forks source link

Inconsistent error messages for edit command #6

Open ThomasHoooo opened 1 year ago

ThomasHoooo commented 1 year ago

image.png

When I want to edit an client out of index, it shows at least one field has to be provided. However, when I parse in edit 0, it shows invalid index.

Expected: the error should be invalid index as well

nus-pe-bot commented 1 year ago

Team's Response

edit 13 has two problems:

so is edit 0.

this app only shows one of the problems.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This causes minor inconvenience to the user because the error messages are not consistent when both of them have the same issues. As a user, I would be confused when I encounter such error messages because there is no consistency. Feel free to test this product out, when you try to type in edit 0 and edit 100, the error messages are drastically different when both of them are just having the same error.


:question: Issue type

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

Reason for disagreement: [replace this with your explanation]