nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Phone input field not flexible enough #140

Open nus-se-script opened 1 day ago

nus-se-script commented 1 day ago

Screenshot 2024-11-15 at 5.26.11 PM.png

Steps To Reproduce

1. Enter command: update 1 p/+65 1234 5678 (HP) 1111-3333 (Office)

Expected

Allowed, as a patient may have multiple phone numbers for different purposes. In some cases, phone number of emergency contacts/next of kin may be relevant for healthcare purposes. These formats are common and valid.

Actual

Not allowed. error message given that restricts input to numbers only.


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

Jaynon commented 4 hours ago

Team's Response

We agree that allowing multiple phone numbers add value to the app, but we do not think that it warrants a medium severity as it is a minor inconvenience as the user can enter the extra phone numbers as a tag.

image.png

Duplicate status (if any):

--