search

wamps-jp / pe

0 stars 0 forks source link

`undo` and `redo` commands interaction with `import` is not made clear in UG #11

Open wamps-jp opened 11 months ago

wamps-jp commented 11 months ago

image.png

As seen in the UG, the list of commands redo (and undo, not pictured) can interact with is made clear. However, during my testing, I found that undo and redo can also revert changes made by importing a different json file. I believe making it clear that undo and redo can also affect import is important, since it may affect users' experience.

nus-pe-bot commented 11 months ago

Team's Response

Thank you for your input. We chose the word "include" as we wanted to show that it was non-exhaustive. In addition, in the UG we stated that any command that modifies the data can be undone and redone, and since importing data falls under this, we believe that this should not be considered a documentation bug as it is already clear to the user.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The phrasing "include" is vague. The purpose of the UG is to have clarity for the user, and having to make an assumption about the interaction of undo with import places the onus on the user to know how your app works, when the responsibility lies on the creators.