nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

ADD Feature does not account for spacing validation for drug allergies #4556

Open nus-pe-bot opened 2 months ago

nus-pe-bot commented 2 months ago

Steps to reproduce:

I have also inputted the following command add ic/S9974567D n/John Doe g/M b/10-07-1999 p/98765432 e/johnd@example.com d/Penicillin|Cephalosporins i/Infectious Disease i/Genetic Disorders

Expected:

Actual:

Screenshots:

image.png

This is problematic as a contact card can get messy especially if users can decide what delimiters they want to use. You can consider supporting this for better convenience and usability for your users. One way you can do is by pre-deciding a delimiter, or handling it like how illness is handled.


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

jmsandiegoo commented 2 months ago

Team's Response

Duplicate of #5429 as this issue mainly address about validation concerns of drug allergy field.

Duplicate status (if any):

Duplicate of #5249