nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Search function unclear #1092

Open nus-pe-bot opened 5 years ago

nus-pe-bot commented 5 years ago

Here is the original list:

image.png

when I type search l/pgp nothing appears. The user guide did not state that it is case-sensitive.


[original: nus-cs2103-AY1920S1/pe-interim#1101]

ngzhaoming commented 5 years ago

Team's Response

First of all, thank you very much for the issue reported. We have taken some time to review and test our respective features again and is safe to say that this function is working as expected.

The main functionality of the search command is to look through the list and find events with the specified keywords. In this case, the specified keyword used is "pgp". Hence, the ASCII value being provided is 112-103-112. Which none of the locations gave the specified ASCII pattern. The closest being "PGP" which bears the ASCII value patterrn of 80-71-80. In this case, each character differs with the original search condition by a value of 32. However, as mentioned in the user guide:

Screenshot 2019-11-16 at 9.18.48 PM.png

The search command searches for specific keywords which means that the search keyword must match exactly with the fields that are given.

As a result, we will reject this issue report based on the aforementioned reasons and also on the following grounds:

Screenshot 2019-11-16 at 8.53.09 PM.png

Once again, thank you for taking the time to test our product and clarify your doubts when using our product. Perhaps next time, to be clearer, we can say that the search function must be ASCII character value specific. Looking forward, we hope to improve and serve you better. :)

Duplicate status (if any):

--