search

nhzaci / pe

0 stars 0 forks source link

Does not specify that edit mode is required for command #9

Open nhzaci opened 3 years ago

nhzaci commented 3 years ago

Screenshot 2021-04-16 at 3.04.10 PM.png

nus-pe-bot commented 3 years ago

Team's Response

First of all, it commands summary, it a page that is supposed to be brief, not details. If we include the condition required to execute save and quit command, does that mean we have to include all important condition for the others command? It will cause inconsistency between command!

The user of the command summary were supposed to have a understanding of the command, else they will be lost in this sea of command. It is supposed to serve as a refresher page, not a reminder page.

second, even if they use the command, it only take like 1 second, and it will probably only occur the very first time they use the command and here the result

image.png

I don't see how it is a valid bug and how the severity can be medium.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The quit command conveniently has a header that says 'Quit edit mode' But the save command has no mention of edit mode.

As a user, I would have tried to use save in the application since having a save command would seem as if my data is not automatically being saved in the application without the mention of it only being used in edit mode.

What the error says in the application and how long it takes to type the command is not relevant, since I am reporting a documentation bug.

:question: Issue severity

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

Reason for disagreement: This a documentation bug that could confuse all users. Saving your progress is something that all users would think that they would have to use. This warrants a medium severity at the very least.