ngkaiwen123 / pe

0 stars 0 forks source link

Product detects numbers as invalid input. #1

Open ngkaiwen123 opened 1 year ago

ngkaiwen123 commented 1 year ago

Product detects whitespaces as an invalid input. While this may be intuitive, error messages could be a bit more descriptive by stating that no spaces are allowed. image.png As shown above, when the input is "1 ", "2 " or "3 ", an invalid message is returned. This may confuse the users as to why an invalid error is returned.

nus-se-bot commented 1 year ago

Team's Response

Our instruction in the UI is clearly stated of "enter the number".

Also, the UG clearly states enter "1", "2" or "3" for registering, logging in and exiting.

It is no surprise that the above error occurs when entering "1 ", "2 " or "3 ".

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is not stated in the User Guide that whitespaces should not be included. Only for the passwords is there any mention of whitespacings. It should have been safe for the user to assume that whitespaces will be ignored even when typing 1,2 or 3.

Screenshot (512).png

Screenshot (516).png

Screenshot (515).png


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** This is not a comestic issue but an actual bug. It should be `severity.Low` at the very least.