search

RunjiaChen / pe

0 stars 0 forks source link

Overzealous input verification for phone #6

Open RunjiaChen opened 5 months ago

RunjiaChen commented 5 months 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

soc-pe-bot commented 5 months 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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]