nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Deactivate command works on inactive items #993

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Deactivate command still works on prescriptions that have already been made inactive.

Screenshot 2022-11-11 at 4.46.46 PM.png


[original: nus-cs2113-AY2223S1/pe-interim#1037] [original labels: type.FeatureFlaw severity.Medium]

RiaVora commented 1 year ago

Team's Response

This is allowed: we want the doctor to be able to ensure that they have de activated something. Performing the command twice has no harm, although one possible improvement could be that we alert the doctor it is already de activated.

Duplicate status (if any):

--