nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Overzealous input validation for phone number #3700

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 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.


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

ziiqii commented 6 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.

Duplicate status (if any):

--