JonLamy / pe

0 stars 0 forks source link

Insufficient phone number validation #3

Open JonLamy opened 1 year ago

JonLamy commented 1 year ago

Steps to reproduce

Expected outcome

Actual outcome

soc-pe-bot commented 1 year 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

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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]