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 data deletion for WPJM User Meta #1430

Merged
merged 1 commit into from
Apr 11, 2018
Merged

Conversation

alexsanford
Copy link
Contributor

Contributes to #1362

This PR, on plugin deletion, deletes the user meta associated with WPJM. The following user meta is deleted:

  • _company_logo
  • _company_name
  • _company_website
  • _company_tagline
  • _company_twitter
  • _company_video

Testing

  • First, you may want to back up your data, or copy it to a fresh WordPress installation.

  • Ensure that the tests pass.

  • Set up at least one user and have that user post a job through the frontend form. Be sure to include all company info.

  • Inspect the database and ensure that the user has the usermeta listed above.

  • Delete the WPJM plugin.

  • Verify the WPJM user meta is removed from the database.

  • Please also test on a multisite installation. When the plugin is deleted from the Network Admin, the WPJM user meta should be removed across the network.

@alexsanford alexsanford self-assigned this Apr 9, 2018
@alexsanford alexsanford requested review from jom and donnapep April 9, 2018 19:43
Copy link
Member

@jom jom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@alexsanford alexsanford force-pushed the add/data-deletion-user-meta branch from 1085f94 to ccaa8c1 Compare April 11, 2018 13:11
@alexsanford alexsanford merged commit 1dd1578 into master Apr 11, 2018
@alexsanford alexsanford deleted the add/data-deletion-user-meta branch April 11, 2018 13:37
@jom jom added this to the 1.31.0 milestone Apr 11, 2018
@jom jom mentioned this pull request Apr 18, 2018
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

Successfully merging this pull request may close these issues.

2 participants