Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overzealous input verification for phone #5592

Open
nus-se-script opened this issue Apr 19, 2024 · 1 comment
Open

Overzealous input verification for phone #5592

nus-se-script opened this issue Apr 19, 2024 · 1 comment

Comments

@nus-se-script
Copy link

nus-se-script commented Apr 19, 2024

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
Copy link
Contributor

zoebelle-pang commented Apr 20, 2024

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):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants