xz0127 / pe

0 stars 0 forks source link

Inconsistent error message on invalid ID input #7

Open xz0127 opened 3 years ago

xz0127 commented 3 years ago

When invalid ID input is provided for replace feature, the error message should be related to the invalid id input and describe what should be the correct ID input format. Screenshot 2020-11-13 at 16.57.02.png

nus-se-bot commented 3 years ago

Team's Response

Error message covers what is wrong. It is sufficient.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I may not think the developers gave a valid response to this bug. Firstly, the command input follows the correct input format as stated in the error message. The actual problem is that the input id for replacement is invalid. Therefore, the user may still feel confused after they receive the error messages. Secondly, the error message still does not provide the correct example for the users.

Screenshot 2020-11-20 at 16.06.57.png

The error message provides a sample command replace 1234 i/91234567. However, itself is also not a valid example as the new id is not a valid input.

Screenshot 2020-11-20 at 16.07.12.png

Therefore, this may mislead the users. After consideration, I would also like to revise my report to FunctionalityBug instead of the original FeatureFlaw as the bug in the error message affects the functionality of the app.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]