search

lihka1202 / pe

0 stars 0 forks source link

Error message displayed is faulty #2

Open lihka1202 opened 1 year ago

lihka1202 commented 1 year ago

The error message displayed when only 1 item in the list is fairly faulty.

delete 90
Sorry, your delete command is invalid. Please ensure [INDEX] is a number and is within the boundary > 0 and <= 1!

As inputting any number that is less than 1 but greater than 0, perhaps a number like 0.5 or any other number, could results in a bad error message. This is merely cosmetic, but can be changed to integer instead.

soc-pe-bot commented 1 year ago

Team's Response

The index is clearly used to count the transactions. Hence, common sense should dictate you cannot have index 0.5.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [I disagree. Even if I were to assume that the users have common sense and understand the difference between a number and an integer, the representation of the range in the current format is still bad UX, since this is merely a cosmetic observation]