nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Insufficient phone number validation #1983

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Steps to reproduce

Expected outcome

Actual outcome


[original: nus-cs2103-AY2223S1/pe-interim#1674] [original labels: type.FeatureFlaw severity.Low]

czhongwei commented 1 year ago

Team's Response

Decision

Not In Scope

Reason for NotInScope

We agree that there may not be phone numbers that are 20 digits long, but disallowing phone numbers of any certain length is not absolutely necessary for the application to serve its purpose.

In fact in the addition of the constraint may hinder the user if there is an extreme case of someone needing to type in 30+ digits in any event.

Screenshot 2022-11-12 at 10.35.23 PM.png

Because applying this additional constraint isn't essential for the application to perform well, and does not increase usability significantly compared to the effort required for implementation and does not prevent a user from inputting a valid phone number, we claim that this is NotInScope.

Duplicate status (if any):

--