search

jingyu987 / pe

0 stars 0 forks source link

Invalid index message inconsistent. #3

Open jingyu987 opened 11 months ago

jingyu987 commented 11 months ago

Having invalid indexes such as s/0 will show "The stall index provided is invalid", but for the view item feature having i/0 will show "Item index is not a non-zero unsigned integer."

Although a small issue, i believe the error messages should be consistent such as "The item index provided is invalid." to prevent confusion from users especially since that error message might be confusing to some.

image.png

image.png

soc-pe-bot commented 11 months ago

Team's Response

We disagree that this is a functionality bug as ultimately, the error message correctly lets the user know that it is the index that is entered incorrectly, and our UG also already lets users know the valid inputs for stall and item indexes.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]