nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

No validation for find parameters #5695

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Reproduction instructions

find m/cs5

Expected

Format error about wrong module format, consistent with add

Actual

No results found (which is correct, btw)

The user might expect there to be validation for the inputs for find, as there is one for add and edit students.


[original: nus-cs2103-AY2223S1/pe-interim#5649] [original labels: type.FunctionalityBug severity.Low]

heyzec commented 1 year ago

Team's Response

The reason being our find command is not a full word match, but a partial word match, hence why there is no validation, unlike add and edit commands.

Example: Say if the TA wants to find all the CS coded modules that he teach, he can simply type find/CS

Duplicate status (if any):

--