search

justinlengch / pe

0 stars 0 forks source link

Find command do not work for partial matches #1

Open justinlengch opened 7 months ago

justinlengch commented 7 months ago

image.png

image.png

The UG mentions find command will work for partial matches such as Han will match Hans, hence Ale should match Alex as well, but it does not.

Steps to reproduce

nus-se-script commented 7 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Find Command does not allow for Partial Searches

According to the UG, the search functionality allow for partial matches Screenshot 2024-04-19 at 4.21.51 PM.png

With Alex Yeoh in the database at index one as shown: Screenshot 2024-04-19 at 4.24.03 PM.png

When the command find Ale is used as input: Screenshot 2024-04-19 at 4.25.13 PM.png

Expected: 1 student listed!, since Alex Yeoh is in the database. Actual: 0 persons listed, despite Alex Yeoh being part of the database. Screenshot 2024-04-19 at 4.25.29 PM.png

Given that the find function is most likely the most used command in the app, this bug might cause the utility of the app to decrease plenty since tutors will now have to remember the entire name of the student (e.g. tutor might remember/call a student Bob instead of his full name Bobby) , causing inconveniences to most of the users.

[original: nus-cs2103-AY2324S2/pe-interim#4374] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for pointing this out. We initially wanted to implement this feature, but we changed our minds and had forgotten to update the user guide.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]

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