Skip to content

Add new EventNotificationHandler class for better thin event management - #1881

Merged
xavdid merged 1 commit into
masterfrom
DEVSDK-2869
Aug 19, 2026
Merged

Add new EventNotificationHandler class for better thin event management#1881
xavdid merged 1 commit into
masterfrom
DEVSDK-2869

Conversation

@xavdid

@xavdid xavdid commented Aug 19, 2026

Copy link
Copy Markdown
Member

Why?

The event notification handler code has been in the beta and private-preview branches since last year, but it's time to take it to GA!

This PR is entirely pre-approved code from previous PRs. It takes the static files from codegen, plus all the manual changes I made directly in beta to get everything working. It's stacked on top of the non-verified work from this week, too. Once this lands, the managed handler code should be in sync across all channels.

Notably, the static files that we used to share functionality are no longer sourced from codegen. Instead, they're now true manual files in master whose changes will flow into preview branches like normal.

What?

  • add EventNotificationHandler class & supporting infrastructure (ability to copy a StripeClient, etc)
  • add stripeClient methods for creating a handler bound to that client
  • add example
  • add tests

See Also

Changelog

Comment thread examples/event_notification_handler_endpoint.py Dismissed
@xavdid
xavdid marked this pull request as ready for review August 19, 2026 23:04
@xavdid
xavdid requested a review from a team as a code owner August 19, 2026 23:04
@xavdid
xavdid requested review from kidus-stripe and removed request for a team August 19, 2026 23:04
@xavdid
xavdid enabled auto-merge (squash) August 19, 2026 23:04
@xavdid
xavdid merged commit 188dce6 into master Aug 19, 2026
18 checks passed
@xavdid
xavdid deleted the DEVSDK-2869 branch August 19, 2026 23:45
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.

3 participants