Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete cascade #53

Closed
belguinan opened this issue May 4, 2018 · 1 comment
Closed

Delete cascade #53

belguinan opened this issue May 4, 2018 · 1 comment

Comments

@belguinan
Copy link
Contributor

Hello,
In databases it's great if you make the delete on cascade on all foreign keys,
for exemple if you make a new customer and a new address, once you destroy the customer; Addresses page doesnt work anymore, It throws an exception so you need to delete the address manually

@jsdecena
Copy link
Owner

jsdecena commented May 7, 2018

In an enterprise level, we do not hard delete anything. Come to think when this customer has an order and you delete it, the history will be gone and you will have an orphaned order. The same way with the addresses, you need to keep backtrack all your history way from when you start your store. It can have a soft delete though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants