Skip to content

add webhook handler guide - #116

Merged
cb-alish merged 1 commit into
v4from
chore/add-webhook-guide
Aug 13, 2026
Merged

add webhook handler guide#116
cb-alish merged 1 commit into
v4from
chore/add-webhook-guide

Conversation

@cb-karthikp

@cb-karthikp cb-karthikp commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Add webhook usage guide in readme

Added README documentation for WebhookEventHandler, including typed event callbacks, webhook handling, optional Basic Auth, fallback behavior, and WebhookResult semantics. Added the handler to the SDK features list.

@snyk-io

snyk-io Bot commented Aug 13, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Secrets 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 400e8f6b-1d91-460b-95c7-a2b5ba50f210

📥 Commits

Reviewing files that changed from the base of the PR and between 26d271e and 66b475a.

📒 Files selected for processing (1)
  • README.md

Walkthrough

README.md now documents WebhookEventHandler, including typed callbacks, optional Basic Auth, fallback behavior, request outcomes, and WebhookResult methods. The SDK feature list now includes webhook event handling.

Changes

Webhook handler documentation

Layer / File(s) Summary
Document webhook handling
README.md
Documents typed customer and subscription callbacks, webhook request processing, success and failure handling, optional Basic Auth, fallback callbacks, WebhookResult methods, and the SDK feature-list entry.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Mergeability Score: ⚪ Minimal · up to 66b47

This PR adds webhook usage guidance to the README without changing product behavior or runtime code, so no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Comment @coderabbitai help to get the list of available commands.

@cb-karthikp
cb-karthikp marked this pull request as ready for review August 13, 2026 04:06
@cb-alish
cb-alish merged commit ef665cd into v4 Aug 13, 2026
9 checks passed
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