Skip to content

fix(stripe): correct payment success redirect url#216

Merged
Vigneshsekar merged 1 commit into
frappe:developfrom
aerele:stripe-payment-success-redirect
May 26, 2026
Merged

fix(stripe): correct payment success redirect url#216
Vigneshsekar merged 1 commit into
frappe:developfrom
aerele:stripe-payment-success-redirect

Conversation

@ravibharathi656

Copy link
Copy Markdown
Member

Issue:

Stripe payment success redirect url is built incorrectly when redirect_to is empty. The flow appends redirect_to=None to the success url, which can break the page or lead to an invalid follow-up redirect. This change updates the redirect logic to append redirect_to only when it has a real value, preserving a valid success url.

Ref: #68040

Backport needed: v16, v15

@ravibharathi656 ravibharathi656 force-pushed the stripe-payment-success-redirect branch from acc3e65 to 0034cf6 Compare May 22, 2026 11:23
@ravibharathi656 ravibharathi656 marked this pull request as ready for review May 22, 2026 11:23
@Vigneshsekar Vigneshsekar merged commit 07ea079 into frappe:develop May 26, 2026
6 checks passed
@Vigneshsekar

Copy link
Copy Markdown
Member

@mergify backport to version-16 and version-15

@mergify

mergify Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

backport to version-16 and version-15

❌ No backport have been created

Details
  • Backport to branch to failed

GitHub error: Branch not found

GitHub error: Branch not found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants