search

nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Search should use no prefix #5845

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Search only requires an argument, which is the query. Hence, it should not use any prefix to better optimize the typer

[original: nus-cs2103-AY2021S1/pe-interim#5824]

Rogerlys commented 3 years ago

Team's Response

Hi there. Thank you for your bug report!

We actually did consider the idea of discarding the argument for query-string prefix. However, there are some reasons we have considered carefully and then eventually decided to keep this new prefix. Firstly, we want our commands have consistent system of format, only INDEX (which is a non-zero unsigned number) will not need prefix, other fields will require the existence of prefixes. We believe this consistency enhances users' experience a lot. Second, we left room for modication and extension for further improvement in the future. What if we want to add more prefixes to the search command to make it more functional to the users for the next milestone (CS2103T onwards)? In this case, we cannot swap the supply of the inputs if there is no query-string prefix.

Therefore, we decide to reject this bug report. Cheers!

Duplicate status (if any):

--