-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add Upi intent flow #2673
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 7dfbf16 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Size Change: +13.2 kB (+1.15%) Total Size: 1.16 MB
|
set selected app only when needed
31acaa2
to
58d188c
Compare
da0597c
to
65ab087
Compare
65ab087
to
35d64e7
Compare
…wait for the upi intent apps instead of redirect
35d64e7
to
f265ee1
Compare
873d06c
to
59a09b8
Compare
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Summary
UPI with the
upi_intent
flow:For mobile:
apps
are returned in /paymentMethods responseIf the
apps
list is returned, we show them in a radio group, we always show the vpa input as the last option.If the
apps
list is not returned or empty, we fall back to the old vpa flow.upi_intent
apps triggersawait
action. We should redirect the shopper to the corresponding apps. The redirect url comes from the BE action response.For non-mobile:
apps
in /paymentMethods response)For UPI in the drop-in:
upi
tx_variant is found.Test UPI component locally
Summary
section for more detailsapps
list.Tested scenarios
Added tests
Fixed issue: COWEB-1353