Skip to content

Revoke and Remove Tokens on Uninstall#133

Open
n7studios wants to merge 4 commits intorevoke-remove-tokens-disconnectfrom
revoke-remove-tokens-uninstall
Open

Revoke and Remove Tokens on Uninstall#133
n7studios wants to merge 4 commits intorevoke-remove-tokens-disconnectfrom
revoke-remove-tokens-uninstall

Conversation

@n7studios
Copy link
Copy Markdown
Contributor

Summary

When the user uninstalls the Kit for WPForms Plugin from their WordPress Site:

  • Revokes all access and refresh tokens by calling the oauth/revoke endpoint
  • Removes all v3 API Key, v3 Secret, v4 Access Token, v4 Refresh Token and v4 Token Expires settings from the database.

https://developer.wordpress.org/plugins/plugin-basics/uninstall-methods/ defines "Uninstall" as:

A plugin is considered uninstalled if a user has deactivated the plugin, and then clicks the delete link within the WordPress Admin.

Testing

  • testPluginDeletionRevokesAndRemovesTokens : test that uninstalling (deleting) the Plugin revokes and removes tokens.

Checklist

@n7studios n7studios added the enhancement New feature or request label Apr 10, 2026
@n7studios n7studios changed the title Revoke remove tokens uninstall Revoke and Remove Tokens on Uninstall Apr 10, 2026
@n7studios n7studios self-assigned this Apr 10, 2026
@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team April 10, 2026 11:42
@n7studios n7studios marked this pull request as ready for review April 10, 2026 11:42
*/
public function deleteKitPlugin($I)
{
$I->deleteThirdPartyPlugin($I, 'integrate-convertkit-wpforms');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The comment mentions no errors output, but I don't see a check for that here or in the method itself

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants