Skip to content

Uninstall: Update Token Revokation#1132

Open
n7studios wants to merge 2 commits into
mainfrom
revoke-token-add-token-hint
Open

Uninstall: Update Token Revokation#1132
n7studios wants to merge 2 commits into
mainfrom
revoke-token-add-token-hint

Conversation

@n7studios

@n7studios n7studios commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Updates the token revokation by using token_type_hint and setting the request Content-Type to application/x-www-form-urlencoded, based on the developer docs at https://developers.kit.com/api-reference/oauth-token-revocation

Testing

Existing tests pass.

Checklist

@n7studios n7studios self-assigned this Jul 4, 2026
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

1 similar comment
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team July 4, 2026 07:53
@n7studios n7studios marked this pull request as ready for review July 4, 2026 07:53
$I->deleteKitPlugin($I);

// Allow the uninstallation routine time to complete.
$I->wait(10);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

On occasion the test fails as it doesn't give WordPress enough time to complete uninstall.php. This bump of the wait seems to resolve.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant