Skip to content

Conversation

@lizkenyon
Copy link
Contributor

Unblock users experiencing TypeError when registering webhook handlers due to Sorbet type checking introduced in shopify_api-16.0.0

Changed to for ShopifyAPI::Webhooks::WebhookHandler in all webhook job templates. This makes the class object itself satisfy the WebhookHandler type expected by
WebhooksManager.add_registrations

Affected templates:

  • app_uninstalled_job.rb.tt
  • customers_data_request_job.rb.tt
  • customers_redact_job.rb.tt
  • shop_redact_job.rb.tt
  • webhook_job.rb.tt

What this PR does

Reviewer's guide to testing

Things to focus on

Checklist

Before submitting the PR, please consider if any of the following are needed:

  • Update CHANGELOG.md if the changes would impact users
  • Update README.md, if appropriate.
  • Update any relevant pages in /docs, if necessary
  • For security fixes, the Disclosure Policy must be followed.

   Unblock users experiencing TypeError when registering webhook handlers due to Sorbet type checking introduced in shopify_api-16.0.0

   Changed  to  for ShopifyAPI::Webhooks::WebhookHandler in all webhook job templates. This makes the class object itself satisfy the WebhookHandler type expected by
   WebhooksManager.add_registrations

   Affected templates:
   - app_uninstalled_job.rb.tt
   - customers_data_request_job.rb.tt
   - customers_redact_job.rb.tt
   - shop_redact_job.rb.tt
   - webhook_job.rb.tt
@lizkenyon lizkenyon requested a review from a team as a code owner January 6, 2026 20:12
@lizkenyon lizkenyon merged commit 6c1c2ec into main Jan 8, 2026
7 checks passed
@lizkenyon lizkenyon deleted the fix-webhook-generator-issue branch January 8, 2026 20:56
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