harin0826 / pe

0 stars 0 forks source link

A person cannot have more than one drug allergy command #4

Open harin0826 opened 1 year ago

harin0826 commented 1 year ago

In real case scenario, a person may have more than one drug allergy.

More than one entries of d/ command during add or edit will be ignored and only the last one is taken.

To get around, users have to split them up with blank spaces all within one d/ and it could lead to confusion.

nus-se-bot commented 1 year ago

Team's Response

It is stated in our User Guide that d/DRUGALLERGIES does not have ellipsis at the end by design choice. However, it is a valid concern and something we would look into in future iterations.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Thank you for responding.

However, it is not stated anywhere in DG or UG that it was left out intentionally because of design choice. From the fact that you chose to make the term plural (drug allergies) instead of singular (drug allergy), I have understood that it was originally intended to assign multiple drug allergies to a patient. Furthermore, there is no explanation provided anywhere about the design choice you mentioned, so I am facing trouble understanding your intent. The examples provided in the documents also do not show a way to enter multiple drug allergies for a patient.

With this problem hindering program usage and its explanation or workaround not given to users, I believe this is a valid feature flaw.

You may suggest that users can do a workaround by adding comma or any other punctuations to separate those (which is my own suggestion because those are not provided in any of the documents, even under the planned enhancements). But doing a short research I found that these allergies are often written with their own punctuations:

image.png

Without providing a way to enter these allergies separately within the system (your tag function works similarly so it is possible), I believe this will remain as a feature flaw.

I'm sure you have done extensive medical research for the design considerations since this is your own program, so let me know if there are any discrepancies between my understanding and current practices in clinics.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** The population with multiple drug allergies is not very big. However, the outcome could be quite severe if they are not accounted for during the treatment. Here is the MOH's national drug allergy [reporting guidelines](https://www.moh.gov.sg/docs/librariesprovider4/default-document-library/national-drug-allergy-reporting-guidelines-2018.pdf): ![image.png](https://raw.githubusercontent.com/harin0826/pe/main/files/38570c26-13f5-46cd-8de8-6bc38cd824d1.png) There are many types of drug allergies and the current system is unable to provide a standard way to indicate multiple entries of those. The worst case outcome may not be frequent but still detrimental, hence I believe the severity should be `Medium`.