nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Unable to remove allergy and vaccination tags from patients. Can only replace with new ones. #3728

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

If a patient with no vaccination prior is accidentally added with a vaccination tag, I cannot remove it unless I delete the person and add the person again with the correct tags.


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

francisyzy commented 1 year ago

Team's Response

In the UG: https://ay2223s2-cs2103-f11-3.github.io/tp/UserGuide.html#edit---edit-a-patient

it is mentioned that --set true needs to be present to set the value. UG.png

Here is an example to show that you can remove invalid vaccination from patients without deleting the patient

Patient with wrong vax.png

You can then run this command to remove all vaccinations from the patient. pat edit 1 --v --set true

Run this command.png

Which will result in a patient with no vaccinations. Removing his or her invalid vaccines.

Fixed patient.png

Therefore you can fix the patient's vaccination without deleting the patient.

This is similar to replacing the list with an empty list.

This is also applicable to Allergies.

Duplicate status (if any):

--