Handle payout.created and payout.reconciliation_completed Stripe webhook events#1461
Conversation
…ook events Add both payout event types to the ignoredEvents array so they are acknowledged without error, consistent with other non-actionable events. Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Stripe webhook handler route adds two event types ( ChangesStripe webhook event handling
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR adds
Confidence Score: 5/5This is a safe, minimal change — two Stripe event types are added to an ignore list with no side effects on existing business logic. Both added event types are compile-time validated by the No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Stripe POST /webhooks] --> B{Verify signature}
B -- invalid --> C[400 Bad Request]
B -- valid --> D{event.type?}
D -- payment_intent.succeeded / ONE_TIME --> E[Record purchase + send receipt email]
D -- payment_intent.payment_failed / ONE_TIME --> F[Send payment failed email]
D -- charge.dispute.created --> G[Send Telegram chargeback alert]
D -- subscriptionChangedEvents --> H[Sync subscriptions / upsert invoice]
D -- ignoredEvents --> I[Return 200 - no-op]
D -- unknown --> J[StackAssertionError]
Reviews (1): Last reviewed commit: "Handle payout.created and payout.reconci..." | Re-trigger Greptile |
Add
payout.createdandpayout.reconciliation_completedto theignoredEventsarray in the Stripe webhook handler so they are acknowledged without throwing an error.Link to Devin session: https://app.devin.ai/sessions/6bedec87599349d9804ca0fd52db55ae
Requested by: @N2D4
Summary by CodeRabbit