search

godjuansan / pe

0 stars 0 forks source link

User Guide Bug : `delete` #20

Open godjuansan opened 3 years ago

godjuansan commented 3 years ago

No details specified that INDEX should not be bigger than the number of contacts and it is not explained what happened when INDEX that is out of bounds is keyed in.

Screenshot from 2021-04-16 15-32-17.png

nus-pe-bot commented 3 years ago

Team's Response

As stated in the second point: The index refers to the index number shown in the displayed person list. This clearly means the index should not be bigger than the number of contacts.

It is obvious that an error would occur if an index that is out of bounds is keyed in. If this occurs in the app, an explanation for the error is provided as the description of the usage of the command is not as detailed there, but in the UG it is detailed enough for the user to deduce the outcome of any erroneous input.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree.

The index refers to the index number shown in the displayed person list does not mean that the only index the user is allowed to key in. In any case, you should clearly explain what happened when the index is out of bounds. It seems to me that you are just trying to search for self-justification rather than disagreeing with rational reasons