nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Feature Flaw - Lack of Confirmation Prompt for Delete and Clear Commands #3138

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

Currently, the Delete and Clear commands function immediately, without allowing users to verify whether they want to proceed with this command. As there is no undo functionality implemented as well, these commands are effectively irreversible. TA's using delete and clear accidentally would not have any ways of retrieving the deleted or reversing the change.

As such, there should be confirmation checks for users, particularly so for delete, which is a command that is frequently used, such that TA's would be able to ensure that they have entered the correct input.


[original: nus-cs2103-AY2425S1/pe-interim#1799] [original labels: type.FeatureFlaw severity.High]

Oceankoh commented 1 week ago

Team's Response

Hi, thanks for the report. We understand the importance of preventing accidental deletions, especially for commands like Delete and Clear. However, the current implementation aligns with our design scope, which prioritizes simplicity and efficiency for experienced users who frequently interact with the system. Introducing confirmation checks or undo functionality would require significant changes to the command flow and is beyond the scope of this project.

Additionally, given that the app is cli based, it is unlikely that the user can accidentally type delete or clear.

Duplicate status (if any):

--