search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

`deletem` allows deletion of medical information from contacts without medical information #3168

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month ago

Screenshot 2024-04-19 at 4.54.00 PM.png

As the title states. In this case, deletem ic/M7654321J simply shows a success message even though the contact with that IC had no prior medical information.

The functionality works nicely and is flexible and not overzealous (does not throw an error), which is good. But perhaps an appropriate alert / warning would be better.

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

javierng2knus commented 4 weeks ago

Team's Response

No details provided by team.

Duplicate status (if any):

--