dillontkh / pe

0 stars 0 forks source link

Sorting resets the filtered list of patients #12

Open dillontkh opened 2 months ago

dillontkh commented 2 months ago

Steps to reproduce:

  1. Use findt to search for a tag that more than 2 people have
  2. Use sort to sort the results according to name

Expected: The command sorts the results of my find operation

Actual: It resets my search results and sorts the entire list

soc-se-bot commented 2 months ago

Team's Response

Hello, thanks for pointing that out, however. in the UG it is stated that the sort command will sort all existing patients in patient list meaning that all existing patients in PatientSync will be sorted instead of the currently displayed patient list. Adding the all existing patients would be intuitive enough for users to infer that patient list is referring to the entire patient list in PatientSync instead of what was displayed.

A more specific phrasing would be to rephrase the UG would be to replace the word patient list to PatientSync to avoid confusion. Thus changing this to a documentation bug. Furthermore, find result usually will not show many patients thus will be uncommon for users to use sort command after find command, which is why changing severity to low.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


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