AmosChan97 / pe

0 stars 0 forks source link

Spacing not supported #3

Open AmosChan97 opened 5 years ago

AmosChan97 commented 5 years ago

image.png

nus-pe-bot commented 5 years ago

Team's Response

The User Guide did not mention that Ducats’ commands can support extra spaces between its parameters. In fact, for each command, the User Guide gives a specific Format for the users to follow. To address the command specific to the tester’s screenshot, please refer to page 4 of the User Guide.

For convenience, the screenshot of the relevant extract of the User Guide is below: image.png

As can be seen, the Format clearly shows that for the new command, there is only 1 space between each of its parameters.

For details on the Formats of Ducats, please refer to page 3 of the User Guide. For convenience again, the screenshot of the relevant extract of the User Guide is below: image.png

From the above extract, coupled with the provided Formats for each and every Ducats command, it should be clear to the user that Ducat’s commands cannot support extra spacing between parameters.

Therefore, the fact that Ducats commands does not support extra spaces between its parameters is the intended behaviour.

The severity of this is changed to low as well. This is because this issue does not affect normal operations of Ducats. In addition, it appears only in very rare situations and causes a minor inconveniences only. The bug type is changed from a FunctionalityBug to a FeatureFlaw as it is not a flaw in how the product works, but lacking in functionality of the product in a that the feature becomes less useful on rare occasions.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

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

Reason for disagreement: [replace this with your reason]


:question: Issue type

Team chose [type.FeatureFlaw]. Originally [type.FunctionalityBug].

Reason for disagreement: [replace this with your reason]