shaelynl / pe

0 stars 0 forks source link

Change listpool to listPool for consistency (Command summary in UG) #11

Open shaelynl opened 3 years ago

shaelynl commented 3 years ago

Maybe can change this to listPool instead, for consistency with the rest of the listPools and findPool below this. Capture.PNG

nus-pe-bot commented 3 years ago

Team's Response

Hi tester,

Thank you for reporting this bug. However, it is a duplicate of #1753, it is referring to the same capitalization issue in the command summary

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Inconsistency in command capitalization

Inconsistency seen in examples and command action name in Command Summary section of UG

Putting this as high severity as the command findPool is not case insensitive and is listed as example. Using the example results in Unknown command error.

image.png

image.png


[original: nus-cs2103-AY2021S2/pe-interim#1753] [original labels: severity.High type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Dear tester, Thank you for finding this error. We have decided to accept this bug, however, we have decided to downgrade this to a "Low" severity bug as it is merely just a documentation bug typo within the command summary. The documentation at the UG features section is accurate and the user should be able to use the product with minor inconvenience.

image.png

image.png

Approaching from the perspective of a user, when he/she realizes that the example is wrong, they would go to the features section to further clarify.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]


:question: Issue severity

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

Reason for disagreement: [replace this with your reason]