Skip to content

fix(onramp): dismiss Google Pay sheet on commit_error#742

Merged
bmc08gt merged 1 commit into
code/cashfrom
fix/dismiss-gpay-on-commit-error
May 20, 2026
Merged

fix(onramp): dismiss Google Pay sheet on commit_error#742
bmc08gt merged 1 commit into
code/cashfrom
fix/dismiss-gpay-on-commit-error

Conversation

@bmc08gt
Copy link
Copy Markdown
Collaborator

@bmc08gt bmc08gt commented May 20, 2026

When a Coinbase commit_error arrives while the Google Pay GMS Activity is on top, nothing dismissed it — the user had to close it manually. Extract the CLEAR_TOP intent trick into dismissPaymentSheet() and call it from wrappedOnPaymentFailure whenever the payment sheet was presented.

When a Coinbase commit_error arrives while the Google Pay GMS Activity
is on top, nothing dismissed it — the user had to close it manually.
Extract the CLEAR_TOP intent trick into dismissPaymentSheet() and call
it from wrappedOnPaymentFailure and wrappedOnCancel whenever the
payment sheet was presented.

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt self-assigned this May 20, 2026
@bmc08gt bmc08gt force-pushed the fix/dismiss-gpay-on-commit-error branch from 297430a to 0e1e760 Compare May 20, 2026 14:00
@bmc08gt bmc08gt changed the title fix(onramp): dismiss Google Pay sheet on commit_error or cancel fix(onramp): dismiss Google Pay sheet on commit_error May 20, 2026
@github-actions github-actions Bot added type: fix Bug fix area: onramp Deposit, purchase, Coinbase, fiat on-ramp labels May 20, 2026
@bmc08gt bmc08gt merged commit 6cdc051 into code/cash May 20, 2026
3 checks passed
@bmc08gt bmc08gt deleted the fix/dismiss-gpay-on-commit-error branch May 20, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: onramp Deposit, purchase, Coinbase, fiat on-ramp type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant