UnwilledTangent / pe

0 stars 0 forks source link

Restricting Drug Allegy field to alphanumeric only makes it hard to add more than 1 drug allegy! #5

Open UnwilledTangent opened 1 year ago

UnwilledTangent commented 1 year ago

image.png

I do not agree that the drug allegy should only be alphanumeric. Doing so will make it difficult to add multiple drug allergies. I would suggest reusing the code used in AB3 for tags, so that multiple drug allergies can be added like a tag! :)

nus-se-script commented 1 year ago

Team's Response

Thank you for the suggestion on the usage of tags, we will certainly take that into consideration. The rejection of commas does not affect usability (all you need are spaces between) and is exactly how we intended to implement it.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I would like to argue against the point that you made, that "all you need are spaces between". Take the drug "Diphtheria Tetanus Pertussis Vaccine" for instance. If all I need are spaces, how can I tell that "Diphtheria Tetanus Pertussis Vaccine" is a singular drug allergy, and not multiple drug allergies, like "Diphtheria", "Tetanus", "Pertussis", "Vaccine"?