search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Overzealous input verification for phone #5592

Open nus-se-script opened 1 month ago

nus-se-script commented 1 month ago

image.png

Description

If someone wants to add two numbers, it throws an error

Steps to reproduce

  1. Launch the application for the first time to load initial data.
  2. Run the add command with 2 phone numbers

    Expected behaviour

    Input should be accepted

    Actual behaviour

    error message is shown

    Reason for severity

    It can be worked around quite easily

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

zoebelle-pang commented 4 weeks ago

Team's Response

Thanks for your issue. The add command for p/ parameter is working as intended. The output message correctly produced an error that indicates that only numbers are accepted but the input you have provided is 'p/1234 5678 (HP) 1111-3333 (Office)' consisting of words and symbols. Also, it was never mentioned in our User Guide that multiple phone numbers can be added.

Duplicate status (if any):

--