search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

deletecheese is allowed when user is not listing cheeses #2967

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

deletecheese is allowed when user is not listing cheeses. When the app is not showing cheeses, the user is still able to deletecheese, I thought that this should not be allowed as deleting is permanent and the information will be lost. When the app is not showing cheeses, the user is basically deleting at random

[original: nus-cs2103-AY2021S2/pe-interim#2967] [original labels: severity.Medium type.FeatureFlaw]

weixue123 commented 3 years ago

Team's Response

This is a feature to provide convenience for the users.

The users might want to switch views to check something else before actually entering the deletecheese command.

The following is a realistic example:

  1. User wants to delete a Gouda at index number 5.
  2. Before deleting the Gouda, the user wants to check if there are any unfulfilled Gouda orders. If there are any, then the user may want to assign the Gouda instead of deleting it.
  3. The user switches view by entering "listorders".
  4. The user confirms there is no unfulfilled Gouda orders.
  5. The user enters "deletecheese 5" to delete the Gouda, without having to switch back to the cheese view.

Duplicate status (if any):

--