Greg-Tan / pe

0 stars 0 forks source link

Find command also finds persons in the archived list #6

Open Greg-Tan opened 2 months ago

Greg-Tan commented 2 months ago

not sure if this is the intended functionality for your app but your ug does not say which lists the find command finds its persons from. I think find command should only be for the current list of contacts that i am viewing (similar to edit command) because upon enter the find command and your app returning me all the persons that match my keyword from both the archived and main list, I will not know which persons belong to archive and which belong to main because there is no indicator. Hence, I think it is a feature flaw

soc-se-bot commented 2 months ago

Team's Response

Thank you for the feedback! This is definitely a use case that should be explicitly stated as allowed or denied which we have failed to do so. An enhancement that we are planning to do is this, which would mitigate the flaw that you have encountered: Screenshot 2024-04-21 at 9.48.43 PM.png

However, we feel that this issue is not in the scope of the current version, and whether we decide to keep the find feature as is after implementing the above enhancement can be further discussed.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]