search

Zhongli5712 / pe

0 stars 0 forks source link

list command does not check for command extra #6

Open Zhongli5712 opened 1 year ago

Zhongli5712 commented 1 year ago

image.png

No error message display when command "list Doctors" is used, and the result of the command is listing all people in the list. This could lead to user think (in worst case) the command "listDoctors" is bugged and can not use this features. A check and error message here would be reasonable.

nus-se-bot commented 1 year ago

Team's Response

The list command works as expected but the current version does not check for the fields after the "list" word which is stated in the UG that anything after the "list" word is ignored because it is no parameter command. The effort required to address this issue is higher than the scope of the current implementation. The severity level should not be "high" as it does not render the product unusable rather it will just cause a minor inconvenience to the user if they use the command wrongly.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** I agree that it is not in scope, however, it is not uncommon for user to use list and listDoctors function. The severity I think is Medium, given that the UG stated this limitation and user can still use the product.