search

ernestlim8 / pe

0 stars 0 forks source link

Add command phone number #1

Open ernestlim8 opened 3 years ago

ernestlim8 commented 3 years ago

image.png

image.png

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

nus-se-bot commented 3 years ago

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

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]