chuahjiajie / pe

0 stars 0 forks source link

No Warning Before Clearing All Data #6

Open chuahjiajie opened 2 months ago

chuahjiajie commented 2 months ago

image.png

Clearing data is an irreversible process with large consequences. The user should be prompted for some form of confirmation before all data is completely cleared.

nus-se-script commented 2 months ago

Team's Response

No details provided by team.

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 a completely separate issue and has to do with the clear command not the undo or redo commands.

The crux of the issue is that it is far too easy to clear all the data, with it just requiring a user to type the word "clear" into the command box.

Further confirmation should be prompted from the user as this command causes a severe amount of data loss.


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** For the reasons stated in the disagreement section this should be a feature flaw.
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]