sohenze / pe

0 stars 0 forks source link

Clear command: No confirmation issued #4

Open sohenze opened 1 year ago

sohenze commented 1 year ago

Steps to reproduce: clear

Expected output: Users should be warned that doing such a command would instantly remove all data from the application

Actual output: No warning or confirmation requested

image.png

Users may accidentally type the clear command and delete all data from WorkBook in the process. They would be able to undo but it may cause unnecessary panic in users. Not all users may not be aware of the undo command as well.

soc-se-bot commented 1 year ago

Team's Response

We did mention

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

The behaviour is documented in the UG but I believe this is a feature flaw as this can be implemented in a better way without much additional effort. Clearing all data from WorkBook is a very drastic action and I think warning or asking the user for confirmation would be appropriate. If the user mistakenly clears all data, the user should not have to rely on another command for a solution when it can be easily prevented in the first place.