chloeelim / pe

0 stars 0 forks source link

Misleading error message when handling large index values #3

Open chloeelim opened 1 year ago

chloeelim commented 1 year ago

Description

Users may accidentally enter large index values, for instance, delete 11111111111. By the command format usage outlined in the UG, this command follows the command format of delete- however, the error message shown is rather confusing:

"Invalid command format! delete: Deletes a person identified by the index number used in the displayed person list. Parameters: INDEX (must be a positive integer) Example: delete 1"

Because the provided index is a positive integer, but the app does not parse the integer properly. Instead, the app should perhaps prompt that the provided index is not valid (not displayed in the list).

Screenshot

Screenshot 2022-11-11 at 5.03.57 PM.png

nus-pe-script commented 1 year ago

Team's Response

Thank you for pointing this out. However, we believe that this is a result of extreme user behaviour and it should not be a bug.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

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]