这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@lizkenyon
Copy link
Contributor

In preparation for the next major version, the deprecated Handler interface has been removed.

Apps must migrate to WebhookHandler which provides webhook_id and api_version fields in addition to topic, shop, and body. This enables apps to properly handle duplicate webhook events and version-specific processing. All tests and documentation have been updated to reflect the new interface.

🤖 Generated with Claude Code

Description

Fixes #

Please, include a summary of what the PR is for:

  • What is the problem it is solving?
  • What is the context of these changes?
  • What is the impact of this PR?

How has this been tested?

Please, describe the tests that you ran to verify your changes.

Checklist:

  • My commit message follow the pattern described in here
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the project documentation.
  • I have added a changelog line.

In preparation for the next major version, the deprecated Handler interface has been removed.

Apps must migrate to WebhookHandler which provides webhook_id and api_version fields in addition to topic, shop, and body. This enables apps to properly handle duplicate webhook events and version-specific processing. All tests and documentation have been updated to reflect the new interface.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@lizkenyon lizkenyon requested a review from a team as a code owner October 14, 2025 18:14
@lizkenyon lizkenyon requested a review from sle-c October 14, 2025 18:15
@lizkenyon lizkenyon force-pushed the remove-old-webhook-handler branch from 0816756 to 01ca410 Compare October 14, 2025 18:20
@lizkenyon lizkenyon merged commit 37b95e4 into main Oct 21, 2025
9 checks passed
@lizkenyon lizkenyon deleted the remove-old-webhook-handler branch October 21, 2025 16:46
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