tayruxin / pe

0 stars 0 forks source link

White spaces in phone number field #4

Open tayruxin opened 11 months ago

tayruxin commented 11 months ago

Currently, the application rejects a valid phone number input like “9299 3652” which is a valid phone number. It was not stated in the error message or UG that spaces are not allowed. Perhaps the team can consider trimming the white spaces of the phone number. Besides, it is common for user to key in phone numbers in set of 4

image.png

nus-pe-bot commented 11 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

"Invalid phone number : Phone numbers should only contain numbers, and it should be at least 3 digits long"

We believe that a white space is not a number.

"Besides, it is common for user to key in phone numbers in set of 4". Our target audience is NUS computing students, which could possibly meet people outside of Singapore for work reason. It is not standardised then to key in numbers in groups of 4 , or some integer K.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]