search

daknam2001 / pe

0 stars 0 forks source link

In the case of duplicates, deletion of one type of medicine only deletes one instance instead of both. #4

Open daknam2001 opened 2 years ago

daknam2001 commented 2 years ago

image.png

nus-pe-bot commented 2 years ago

Team's Response

Function works as expected.

Since the addmed function accepts duplicates in this current iteration, it would only make sense that the delete function will delete the duplicate instead of all the instances, like it is being suggested in this bug report.

Due to human error, it is possible that the user inputs the same information and wishes to delete it as it is a duplicate. It would make more sense to delete that duplicate, instead of deleting all instances of the specified input and the user having to re-input the same information again.

Hence, to deal with the above, the function is supposed to work this way.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]