Skip to content

Uninstall: Update Token Revokation#1132

Merged
n7studios merged 2 commits into
mainfrom
revoke-token-add-token-hint
Jul 7, 2026
Merged

Uninstall: Update Token Revokation#1132
n7studios merged 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.

@n7studios n7studios merged commit 978743f into main Jul 7, 2026
260 of 276 checks passed
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.

2 participants