nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Do not allow area code for phone numbers #941

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

image.png

using '+' in the phone number for are code is not allowed


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

JoanneHing commented 1 week ago

Team's Response

Thank you for your feedback! But our team would like to clarify that we do accept area code, since we do not restrict the phone number length to be just 8 digits, hence a user is able to enter 6512345678 which includes the area code. However, our app does not accept special characters such as + that is entered here. This constraint is also specified in both the UG and the error message in the product.

Here is a screenshot of our UG for your reference. Screenshot 2024-11-18 at 12.03.22 AM.png

Hence, we would categorize this bug as NotInScope as the UG specified this is not supported and user cannot attempt to use the + sign in the phone number, if you does so, the error message will be shown and guide the user about the correct format.

Duplicate status (if any):

--