nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Special characters recorded as drug allergies in add command #5249

Open nus-se-script opened 2 months ago

nus-se-script commented 2 months ago

image.png

command: add ic/S9974944F n/John Doe p/91234567 e/johndoe@email.com g/M b/11-11-1990 d/|| | | || i/Infectious Diseases Here the separator characters are actually recorded as drug allergy types in the app, leading to confusion. Should consider imposing some restrictions on the type of information to be placed between separators.


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

jmsandiegoo commented 2 months ago

Team's Response

When developing drug allergy functionality, we have decided to allow the user to have the flexibility to input any information related to the drug allergy field. This could include explanation on specific person's reaction for a specific drug allergy, or in rare occasion where clinics has some SOP where they have symbols that represent a respective drug allergy, which in that case they can just enter that symbol in the field. We aim to make it flexible enough for various use cases that various clinics might have. Though, regarding overzealous validation the bug issue can be addressed in by simply adding some warning messages that only symbols are inputted in the future releases. Hence marking this NotInScope

Duplicate status (if any):

--