jiaxinnns / pe

0 stars 0 forks source link

findMedCon, then deleting the medical condition causes filter to be removed #7

Open jiaxinnns opened 2 weeks ago

jiaxinnns commented 2 weeks ago

run findMedCon tooth decay

image.png

run delMedCon i/s1234567a c/tooth decay

filter will be removed without warning and the entire list is displayed again

image.png

soc-se-bot commented 1 week ago

Team's Response

Thank you for your feedback! As indicated in our user guide, any commands (except those in finding patient section), will refresh the panel and show all patients upon successful execution. image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I have categorised it as a feature flaw (not a functionality bug) since even though it is documented:

This makes the find feature only truly useful for viewing the list of patients rather than performing operations on their data. Unlike names where many people are unlikely to have the same name, medical conditions can be quite widespread (eg. flu), so it is likely that the user would want to perform multiple operations after running findMedCon. Thus, I feel that this limitation makes the feature less useful to the intended target user (doctors who have multiple patients with evolving needs).


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]