nichee / pe

0 stars 0 forks source link

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

Open nichee opened 4 months ago

nichee 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

nus-pe-script 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

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Perhaps this could be response.notInScope.

image.png "We state in the instructions ... that all text in capital letters means that input is required from the user. " You simply state that they are inputs, not that they are required inputs.

image.png "make it clear that searching using patient and doctor requires parameters." I do not agree that this is clear - in the image you attached, there is no implication that the parameters are required, only that Upper case refers to parameters that the user can supply