[n7studios]

Summary

Replaces the hourly WordPress Cron event (that imported Broadcasts) with a Kit webhook, so Broadcasts are imported as soon as they're published in Kit. This also removes the reliance on WordPress Cron, which doesn't run reliably on low-traffic sites or when DISABLE_WP_CRON is set.

• Adds a kit/v1/broadcasts/webhook REST API route. It receives post.published events from Kit, verifies the signature, ignores duplicate events and imports new Broadcasts.
• Registers the webhook endpoint in Kit when the Broadcasts import is enabled. Deletes it when the import is disabled, the Plugin is disconnected from Kit or the Plugin is deactivated.
• Settings > Kit > Broadcasts:
  • adds a Webhook field showing the endpoint's status in Kit, with a Register / Re-register webhook button;
  • Import now imports Broadcasts immediately, instead of scheduling a Cron event.
• Removes ConvertKit_Cron and the convertkit_resource_refresh_posts Cron event; the event is unscheduled on Plugin update.

Testing

• BroadcastsWebhookTest: registration, deletion, status, receiving events (valid, duplicate, invalid/expired signature, import disabled) and Plugin update routines.
• BroadcastsToPostsCest: uses Import now instead of running the Cron event.
• BroadcastsToPostsSettingsCest: checks the Webhook field, and that registration fails gracefully on a local site.
• Removed Cron-related tests, the WPCron helper, and wp-crontrol from the test group.

Checklist

• I have written a test and included it in this PR
• I have run all tests and they pass
• The code passes when running the PHP CodeSniffer
• Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
• Security and Sanitization requirements have been followed
• I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

[github-actions]

WordPress Playground

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

Click here to test your changes in WordPress Playground

[noelherrick]

503s are temporary errors - shouldn't this be a 501?

[noelherrick]

How heavy is this call vs. something that imports it one by one or with a list of IDs? We have the post(s) to import