search

CS2103-AY1819S2-W16-1 / main

UniLA
https://cs2103-ay1819s2-w16-1.github.io/main/
0 stars 1 forks source link

Commands needs to be more explicit in its functions #146

Closed nus-se-bot closed 5 years ago

nus-se-bot commented 5 years ago

Command "list" list does nothing. I am not sure what it is doing since the persons are loaded into the address book already. Is there a need for this command?

Suggest Improvement 1) In the event this command is not a very "wow" factor, you can consider deleting it 2) you can have a list by tags since you have a tag for the address book.

E.g. list \t"parents" The command should list all the persons with the tags parents.

list \t"CS2103T classmates" The commmand should list all the persons with the tags CS2103T classmates

Reported by: @jeraldtsy Severity: Medium

[original: nus-cs2103-AY1819S2/pe-dry-run#201]

jwl1997 commented 5 years ago

list is necessary to provide index for delete and select, the feature you demo above is find command that we have already implemented