search

chishanw / pe

0 stars 0 forks source link

Inaccurate error message #4

Open chishanw opened 4 years ago

chishanw commented 4 years ago

When the user types play 0, an error message appears informing the user that the index is invalid. However, the error message says the index must be a positive integer, and 0 is can be considered a positive integer to some people. The error message should be more detailed and informative so as to not confuse the user, such as by telling the user the index must be a positive, non-zero integer.

image.png

nus-pe-bot commented 4 years ago

Team's Response

Thank you for your report.

We will continue with the reasonable and common definition of 0 not being a positive integer. Furthermore, INDEX is obtained from the numbers shown in-app, where 0 is never shown.

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]