nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

clear command does not have warnings #1096

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

In your UG, you mentioned that Prudy is used by Prudential financial agents for client management, to record important information of client's personal details such as their name and phone numbers, as well as their policies and claims.

However, your app has a clear command that instantly wipes the data of your app, with no way to retrieve the lost data. This may dissuade actual financial agents in using your app to store important information.

image.png

image.png

A possible suggestion would be to either remove this functionality, or introduce a confirmation check before clearing, or to introduce a undo feature before introducing this feature.


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

ckaayy commented 1 week ago

Team's Response

Thank you for your bug report! However, the team has already did their best in warning the users to be weary when using the clear command, as shown in the attached screen shot of the user guide below. Furthermore, it has been stated, below the warning box, that we will document on how to edit the data file in future iterations. The team also felt that the issue of accidentally clearing data is very rare considering the warnings in the user guide, and hence has decided to push further changes on the clear command (to include confirmation) to a later iteration. Nevertheless, it is still something the team may consider in the future, and hence, we have listed it as NotInScope.

Furthermore, due to the reasons above, we felt that such issues occurs very rarely and will not affect most users, and thus has decided to move the severity to Low rather than the suggested High. The argument that the lack of confirmation for the clear command will dissuade most users from using the application is not justified. The lack of confirmation should not be a deal breaker to most.

Screenshot 2024-11-18 at 10.51.41 PM.png

image.png

image.png

Duplicate status (if any):

--