nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Deleting a customer does not remove associated orders #3574

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

I'm not sure if this counts as a documentation bug or a functionality bug. However, deleting a customer in FoodOnWheels does not remove orders associated with that customer. This could lead to a build up of orders that have no customers or inconvenience users by forcing them to manually remove all orders for that customer. If there is reason for this behavior, it might be good to document under the corresponding UG section.


[original: nus-cs2103-AY2122S2/pe-interim#3566] [original labels: severity.Low type.FunctionalityBug]

xzzz3 commented 2 years ago

Team's Response

Hello tester, thank you for taking your time to test our app and providing this report! However, this is actually a discussed and considered behaviour of our FoodOnWheels app, that the customers' order should stay in the app after deleting the customers. Consider this: our app is targetted at restaurant delivery managers, and they are interested in how the delivery is doing overtime. If a customer called Aaron made a delivery order in the past, and the manager deleted Aaron (for space saving in the app, or due to accident), the order should remain as it contributes to the past revenue, which is an important statistic to help the restaurant. If the customers' orders are deleted with their deletion, this will result in detrimental loss of information that cannot be recovered, and even make the app unusable for users who like to clean up their interface often by deleting unused customers. This is also a very common practice in delivery that does not require further explanation (as mentioned above, it is so detrimental that it is basically fundamental for any delivery management app to have it).

Therefore, we propose rejecting this issue. Thanks!

Duplicate status (if any):

--