search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Many commands with find functionality #1343

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

Action: Want to be able to search for persons



Expected result/output: Able to do that with a single command that can search for multiple fields



Actual result/output: Have to navigate to different commands to seach for different fields. Have to refer back to the user guide frequently


Suggestions: Could combine the search commands together into one big command. This way users could also filter person with multiple conditions at once.

[original: nus-cs2103-AY2425S1/pe-interim#1855] [original labels: type.FeatureFlaw severity.Low]

flyingsalsa commented 18 hours ago

Team's Response

Due to complexity of adjusting the find command to work with these different kind of input parameters that are nested with a separate object or function completely differently from a string, the decision was made to separate them into different find commands. While definitely a good improvement to have we mark it as NotInScope as all features are still working as intended.

Duplicate status (if any):

--