search

narwhalsilent / pe

0 stars 0 forks source link

Tag related commands cannot be undone #12

Open narwhalsilent opened 4 months ago

narwhalsilent commented 4 months ago

Way to reproduce: tag+ aaa undo

Expected: tag aaa deleted.

Actual:

image.png

Explanation: DG only states that the team aims to add undo features for commands which are modifications of the contact list. Since the tag list is not part of the contact list, this bug is not covered by the planned enhancement.

image.png

Improvement: Allow redo of tag related commands.

nus-pe-script commented 4 months ago

Team's Response

We are rejecting this bug as we mentioned in the redo undo command that tag commands cannot be undone.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I thank the team for pointing out the relevant part of the documentation.

In that case, I believe that this bug is in fact a severity.Low type.FeatureFlaw because the real issue is inaccurate error message shown in the app.

This is because while the error message says there is "no more command to redo", there clearly was a command before it which has not been undone.

In other words, the error message should have explained that the previous tag+ command is not undoable in the current version of the product.