shaelynl / pe

0 stars 0 forks source link

Substrings can be found using find #5

Open shaelynl opened 3 years ago

shaelynl commented 3 years ago

In your UG, you stated that "Only full words will be matched e.g. Han will not match Hans." All substrings can be found though. E.g. Already have a passenger called Charlotte Oliveiro in GME, find n/Char will display Charlotte Oliveiro's profile. Screenshot (53).png

nus-pe-bot commented 3 years ago

Team's Response

Dear tester, thank you for the bug reported!

Unfortunately it seems that as we changed the find feature for searching of names, the behavior of the feature was not updated as well in the user guide. As such, I believe that it would be more appropriate for this to be filed as a documentation bug instead.

Thank you!

The 'Original' Bug

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

Find command matches partial names

Documentation states that only full words will be matched

image.png

Searching for Ale listed Alex Yeow

nameFindBug.gif


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

Their Response to the 'Original' Bug

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

Dear tester, thank you for the bug reported!

Unfortunately it seems that as we changed the find feature for searching of names, the behavior of the feature was not updated as well in the user guide. As such, I believe that it would be more appropriate for this to be filed as a documentation bug instead. Also, as this behavior actually allows for more flexibility, we would like to downgrade the severity of the bug to a low instead as such a behavior only results in minor inconvenience instead.

Thank you!

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 type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your reason]