search

yyccbb / pe

0 stars 0 forks source link

Special characters recorded as drug allergies in add command #8

Open yyccbb opened 2 months ago

yyccbb 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.

nus-pe-script 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

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]