Undocumented breaking change? (stripe_checkout_session_id
vs session_id
)
#1110
Unanswered
marckohlbrugge
asked this question in
Q&A
Replies: 1 comment
-
Yeah, forgot to add that. If you use the |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
40c637f changed the default success URL parameter from
session_id
tostripe_checkout_session_id
.This broke my Pay implementation which had code like this:
Which no longer works, because Pay uses
stripe_checkout_session_id
.The fix is easy. Just switch to using
params[:stripe_checkout_session_id]
, or better yet usePay.sync(params)
.But I was surprised to run into this breaking change which doesn't seem to be documented in the changelog?
I suggest adding it retroactively.
Beta Was this translation helpful? Give feedback.
All reactions