pkpaing / pe

0 stars 0 forks source link

Find feature narrowing down search #5

Open pkpaing opened 1 year ago

pkpaing commented 1 year ago

Screenshot 2023-04-14 at 2.43.37 PM.png

Screenshot 2023-04-14 at 2.46.43 PM.png How to replicate: find cs2101 -> find lecture

In trying to narrow down the search results, e.g. find cs2101 -> find lecture, to display cs2101 lecture as the result, it displays all modules with the lecture tag instead. The user guide does not indicate whether the find command stack on top of each other or generate a completely new set of search results. This provides a minor inconvenience to me as a user who is trying to narrow down the search.

nus-se-script commented 1 year ago

Team's Response

Thanks for the feedback.

This can be implemented in the future.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: As I've mentioned above, the user guide does not indicate whether the find commands generate a completely new set of search results. This provides a minor inconvenience to me as a user who is trying to narrow down the search and as a user who DOES NOT expect a new set of search results to be generated when I enter the Find command on a specific displayed list of contacts. Hence, I feel that this is a severity.Low bug.


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.DocumentationBug`] - [x] I disagree **Reason for disagreement:** As a feature, I think it's fine that your "Find" commands not to stack on top of one another to allow a narrowed down search. Rather I would have expected some documentation on this instead of expecting you to implement a "search narrowing" feature. Hence why I would think that this is a documentation bug and not a feature flaw.