search

dvdweien / pe

0 stars 0 forks source link

deletecheese is allowed when user is not listing cheeses #7

Open dvdweien opened 3 years ago

dvdweien 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

nus-pe-bot 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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]