search

sogggy / pe

0 stars 0 forks source link

undo/redo not working for find #4

Open sogggy opened 3 years ago

sogggy commented 3 years ago

After using find, the undo/redo commands does not have any visual updates to the user that the state has been reverted, if a 'find' command that does not find anything was used.

image.png

image.png

After redoing, still cannot see anything visual updates.

Such a behaviour should perhaps be specified in the user guide, which is not, from what I see!

nus-pe-bot commented 3 years ago

Team's Response

image.png

This is part of the intended functionality of our application. The find command does not change the state of the inventory book or the delivery book and thus does not warrant an undo command. We mentioned this in our Developer guide, but could have made it clearer in our User guide. Hence, we will change this as a documentation bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: This should be in scope, as it is a documentation error that should have been included in the UG.

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]