Skip to content

refactor(stripe): handle webhooks with the basil API version - #431

Merged
xabg2 merged 1 commit into
refactor/stripe-basil-paymentsfrom
refactor/stripe-basil-webhooks
Oct 28, 2026
Merged

refactor(stripe): handle webhooks with the basil API version#431
xabg2 merged 1 commit into
refactor/stripe-basil-paymentsfrom
refactor/stripe-basil-webhooks

Conversation

@xabg2

@xabg2 xabg2 commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator

What's new

  • Paid and failed invoice webhooks now read the invoice with the new Stripe API version, so they keep working whichever API version the webhook endpoint is configured with.
  • Plans and coupons applied after a paid invoice are read from the new invoice format, so users keep getting the right storage and discounts.
  • Refunds and lost disputes now find the related invoice through its payment, so the right subscription is cancelled or the lifetime plan removed.
  • Refunds of charges that don't belong to an invoice are now ignored instead of failing.

Depends on #430

@xabg2 xabg2 added the enhancement New feature or request label Oct 26, 2026
@xabg2 xabg2 self-assigned this Oct 26, 2026
@sonarqubecloud

sonarqubecloud Bot commented Oct 26, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
91.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@xabg2 xabg2 mentioned this pull request Oct 26, 2026
Merged
@xabg2
xabg2 added this pull request to stack #433 October 27, 2026 07:37
@xabg2
xabg2 marked this pull request as ready for review October 27, 2026 09:38
@xabg2
xabg2 requested review from jzunigax2 and sg-gs as code owners October 27, 2026 09:38
@xabg2
xabg2 merged commit 88eaa73 into master Oct 28, 2026
14 checks passed
@xabg2
xabg2 deleted the refactor/stripe-basil-webhooks branch October 28, 2026 08:30

This branch was previously deployed

1 inactive deployment
test — 3fd93843 Deployed Oct 26, 2026 by xabg2 via Run tests (24.11.0) #1181
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jzunigax2 jzunigax2 jzunigax2 approved these changes
@sg-gs sg-gs Awaiting requested review from sg-gs sg-gs is a code owner

Assignees

@xabg2 xabg2

Labels

enhancement New feature or request ready-for-preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xabg2 @jzunigax2