nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Overzealous input validation for phone number #5417

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 months ago

Screenshot 2024-04-19 at 4.41.52 PM.png

Explanation: Program does not allow for phone number with dashes. It is better to warn rather than block such inputs.


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

brennalaurentan commented 6 months ago

Team's Response

It has already been stated in the user guide that only numeric inputs are accepted as a valid parameter for the phone number field.

image.png

Additionally, the screenshot provided does not reflect the bug brought up in text, in the bug report.

Duplicate status (if any):

--