search

chuahjiajie / pe

0 stars 0 forks source link

Undo Does not Work After Clearing All Data #16

Open chuahjiajie opened 2 months ago

chuahjiajie commented 2 months ago

image.png

As seen above, I cannot undo me clearing all data. A user may feel safe to use the clear feature given that he or she can just undo it. However, this is not the case

nus-pe-script commented 2 months ago

Team's Response

"clear" is not an usual command in normal workflow. The user should be warned in the User Guide instead.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Undo and Redo Limitations not Listed Correctly

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

As stated above undo and redo does not work for the link feature.

Additionally it does not work for the clear feature which is not stated anywhere.

[original: nus-cs2103-AY2324S2/pe-interim#3580] [original labels: severity.Medium type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This is an issue that although will not occur too often, it causes a severe and tangible degree of data loss and steps need to be taken to address this issue.

Moreover, this is not a duplicate issue of a documentation bug. Intuitively speaking, there is no reason a user should not expect themselves to be unable to undo any command unless it is specifically stated.

In almost all modern applications even the deletion of an entire file can be undone.

## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** For the reasons stated above, this is an issue with the undo command itself and should be a functionality bug.
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]