Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

White spaces in phone number field #4

Open
tayruxin opened this issue Nov 17, 2023 · 1 comment
Open

White spaces in phone number field #4

tayruxin opened this issue Nov 17, 2023 · 1 comment

Comments

@tayruxin
Copy link
Owner

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
Copy link

[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

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your reason]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants