search

adam-ky / pe

0 stars 0 forks source link

Error message for invalid day does not specify day has to be integer #3

Open adam-ky opened 2 years ago

adam-ky commented 2 years ago

image.png

nus-pe-bot commented 2 years ago

Team's Response

As it is mentioned in the UG that day should be an integer, and the fact that you will refer to the UG for example valid inputs for the command, we do not think this should be construed as a bug.

Hence we reject this bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I absolutely agree with your response and that there is no discrepancy between the UG and the feature. However, following the guidelines in the screenshot below, I think "integers" (instead of "numbers") would be more appropriate as it would be explicit for users. I agree that severity should be severity.VeryLow in this case.

image.png

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]