search

Daviancold / pe

0 stars 0 forks source link

Inconsistency with format of addtwdc command #7

Open Daviancold opened 2 months ago

Daviancold commented 2 months ago

image.png

The format and the example for the addtwdc command is inconsistent, with the format stating that the /on parameter should have a "date", but the example stating that it should be a "day" parameter

Note: This feature, as mentioned in a previous bug, does not work, and hence I was unable to determine which is the correct format

nus-se-script commented 2 months ago

Team's Response

There is an example of the function being used with the word monday, so the user can infer that the Date takes in a day within the week.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As a developer, one should never assume that the user can infer the correct input, especially since there is no way to truly ascertain which one is the correct one through the buggy application.

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** As mentioned in the bug, the feature does not work at all. The severity was chosen to be high because as the user, I am unable to determine which is the correct command input, especially when the user guide contains 2 example formats to use. Perhaps the severity could be lower, but it should not be shrugged off as a cosmetic error and only be `severity.VeryLow`.