nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

expected function of "patient" and "doctor" command #1426

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

image.png A user could reasonably assume that using "patient" or "doctor" with no further terms would show all patients and show all doctors respectively. This however is rejected as an invalid input


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

Alteqa commented 4 months ago

Team's Response

Thank you for raising this concern, however, this command, similar to the rest of the find commands requires parameters for the command to search as clearly mentioned in the user guide. We state in the instructions what is the appropriate format for all commands and that all text in capital letters means that input is required from the user. This is a possible extension that may provide additional features but at the moment our functionality requirements make it clear that searching using patient and doctor requires parameters.

image.png

image.png

Duplicate status (if any):

--