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

Add Cascading Delete Features #883

Closed
5 tasks
joshfeck opened this issue Jan 10, 2019 · 5 comments
Closed
5 tasks

Add Cascading Delete Features #883

joshfeck opened this issue Jan 10, 2019 · 5 comments

Comments

@joshfeck
Copy link
Contributor

joshfeck commented Jan 10, 2019

Moving related "Deletion Behavior Refactor" milestone from codebase. Which has some related tickets for reference:

  • Cascading Deletion options for models 9984

  • Permanently delete any metadata associated with our CPT objects automatically via EEM_CPT_Base when a permanent delete is initiated 9177

  • automatically delete related extra meta when deleting an item 6411

  • Add bulk delete transactions with chained delete of related registrations & payments 11496

  • avoid orphaned data when deleting a WP user that has related events 7961

See also Brent and Darren's comments in https://events.codebasehq.com/projects/event-espresso/tickets/10754

See also the discussion in https://events.codebasehq.com/projects/event-espresso/tickets/10891
where in the end, Mike remarked:

ok I'm going to put this on hold then. This can be incorporated into the cascading deletion code, or invalidated (and the branch removed) once that happens

With regards to a bulk payment deletion feature, I don't think we need to add that. Instead, if we can allow for deleting transactions where all related entities also get deleted, there shouldn't be a need for bulk deleting payments.

See also related GitHub issue #349

@joshfeck
Copy link
Contributor Author

Two further ideas noted by @nerrad , related to allowing contacts to be permanently deleted:

  1. Create an option somewhere in EE for purging the database of records prior to a certain date. That way people would just give the date to purge records for, and we can just delete permanently all related data for that date (that means' some contacts might still remain if there are registrations on that contact that are more up to date).
  2. Create the ui/ux for deleting permanently transactions and work out the issues related to that (how are line items affected? How are attached registrations affected) and once that is worked out, then allow permanently deleting contacts but ONLY if all related transactions/registrations are deleted.

@stale stale bot added the status:stale label Jun 5, 2019
@eventespresso eventespresso deleted a comment from stale bot Jun 6, 2019
@stale stale bot removed the status:stale label Jun 6, 2019
@tn3rb tn3rb assigned tn3rb and mnelson4 and unassigned tn3rb Jul 15, 2019
@tn3rb
Copy link
Member

tn3rb commented Jul 15, 2019

needs :octocat: issues created from all of the original codebase tickets

@tn3rb tn3rb changed the title Add cascading delete features Add Cascading Delete Features Jul 15, 2019
@joshfeck
Copy link
Contributor Author

@garthkoyle garthkoyle modified the milestone: Cascading Deletions Nov 25, 2019
@garthkoyle garthkoyle added this to the Cascading Deletions milestone Dec 6, 2019
@stale stale bot unassigned mnelson4 Apr 4, 2020
@stale stale bot added the status:stale label Apr 4, 2020
@eventespresso eventespresso deleted a comment from stale bot Apr 7, 2020
@stale stale bot removed the status:stale label Apr 7, 2020
@stale
Copy link

stale bot commented Aug 8, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status:stale label Aug 8, 2020
@stale
Copy link

stale bot commented Aug 22, 2020

This issue has been automatically closed because it has been stale for a significant period of time without any activity.

@stale stale bot closed this as completed Aug 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants