search

Hibeom0929 / pe

0 stars 0 forks source link

Overzealous input validation for phone number #3

Open Hibeom0929 opened 2 months ago

Hibeom0929 commented 2 months ago

image.png

For now, only the numbers are accepted and should be more than 3. However, sometimes users might want to take notes for phone numbers like the image given. Also, a person could have multiple phone numbers so the user might want to take notes like 1234(personal), 5678(business). So, it it overly restricted the input.

nus-se-script commented 2 months ago

Team's Response

image.png

We have already stated in the UG that we only accept numerical inputs for phone numbers.

Furthermore, if the user wants to take notes, he can add it as tags.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]