Quasant / pe

0 stars 0 forks source link

Incorrect error message when getting attendance of a student #6

Open Quasant opened 1 week ago

Quasant commented 1 week ago

image.png

nus-se-script commented 1 week ago

Team's Response

image.png

This has been covered in the user guide. The prefix sn has been misspelt in your command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I admit that I had not noticed the typo in the sn prefix, hence why I thought a screen shot would suffice to show that this was a functionality bug.

However, I would like to change my bug type to a feature flaw instead. I was simply utilizing the command as a user would, and did not intentionally put in an extraneous parameter. This is a genuine mistake that can be done by any user, and simply having it in your UG does not negate the fact that a user can be misled by the error message. This can potentially be a notInScope as such since there might be substantial changes needed to implement the better error checking mechanism.