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

Add command phone number #1

Open
ernestlim8 opened this issue Nov 13, 2020 · 1 comment
Open

Add command phone number #1

ernestlim8 opened this issue Nov 13, 2020 · 1 comment

Comments

@ernestlim8
Copy link
Owner

ernestlim8 commented Nov 13, 2020

image.png

image.png

Phone number does not have validation, phone number should not contain more than 8 numbers

@nus-se-bot
Copy link

nus-se-bot commented Nov 18, 2020

Team's Response

This issue is inherited from AB3 as seen replicated below.
According to forum issue #429, this bug report is rejected since it's the same behavior as in AB3.

Address, phone and email validation have been inherited from AB3 and they are for typical use cases. Users are not restricted to certain address/phone/email to cater to wider target audience which may need to handle foreign address or phone. Unnecessary restriction may lead to it being bugged/unusable for some of our intended target audience.

image.png

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


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