nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

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

Open nus-pe-bot opened 1 year ago

nus-pe-bot 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! :)


[original: nus-cs2103-AY2223S2/pe-interim#233] [original labels: severity.VeryLow type.FeatureFlaw]

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

Duplicate status (if any):

--