Skip to content

CryptoOnramp SDK: Adds Ability to Receive L0 KYC Info From Google Pay#12740

Merged
Twigz merged 30 commits into
masterfrom
twigz/google-pay-kyc-fill
Apr 6, 2026
Merged

CryptoOnramp SDK: Adds Ability to Receive L0 KYC Info From Google Pay#12740
Twigz merged 30 commits into
masterfrom
twigz/google-pay-kyc-fill

Conversation

@Twigz

@Twigz Twigz commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds support for automatically pulling KYC information from Google Pay to make it easier for clients/new users to go through the crypto flow.

  • Adds new CompletedWithKycInfo result type when collecting payment that provides clients with a KYCInfo model that they can use to fill out user information.
  • Updates example application to allow gathering of KYC information, and autofills it based on Google Pay being selected.

Motivation

I don’t have a PRD for this, but this comes via request from the crypto team. To open the link in Stripe Slack, paste/send this Lickability Slack link to Slackbot and it should convert it.

Testing

  • Added tests
  • Modified tests
  • Manually verified
  1. Login to the example application.
  2. Authenticate via OTP.
  3. Select Google Pay as a payment option, and complete the google pay flow.
  4. See the updated example app KYC fields be filled with the Google Pay information.

If you want to test uploading KYC information gathered this way, you must make a new account (or use one that has yet to upload KYC info).

Screenshots

KYCSection

Twigz and others added 23 commits March 4, 2026 16:39
@Twigz
Adds in optionality to DOB, and allows for DOB entry
bb2bf43
@Twigz
Update OnrampInteractor.kt
1d5588e
@Twigz
Getting further in testing implementation
1e693ec
@Twigz
Merge branch 'master' into twigz/l0-kyc
f274c8d
@Twigz
Cleanup
4a00773
@Twigz
Add back defaults
6d3a8e7
@Twigz
Merge branch 'master' into twigz/l0-kyc
bcabd6c
@Twigz
Update KYCCollectionRequest to match iOS better
463c0b1
@Twigz
Update KycInfo to all optional to match iOS
7b62f1c
@Twigz
Merge branch 'twigz/l0-kyc' into twigz/google-pay-kyc-fill
e5aa6b7
@Twigz
Just pass back kycInfo
0801203
@Twigz
Merge branch 'master' into twigz/google-pay-kyc-fill
cbcb817
@Twigz
Removes kyc retrieval, passes back kyc regardless
64be500
@Twigz
Small cleanup refactor
4bc20c2
@Twigz
Merge branch 'master' into twigz/google-pay-kyc-fill
848c704
@Twigz
Update OnrampSelectPaymentCallback.kt
64c9079
@Twigz
Merge branch 'twigz/google-pay-kyc-fill' of https://github.com/stripe…
99141b3 
…/stripe-android into twigz/google-pay-kyc-fill
@Twigz
Add KYC first name / last name to the VM for updating it from google pay
e3e94fc
@Twigz
Use the google pay address and submit it instead of the existing stat…
3a9b3d5 
…ic address
@Twigz
lint
afdc143
@Twigz
Update OnrampActivity.kt
29d2a4d
@Twigz
Don't return kyc info if it's all null/empty
8471c88
@Twigz
Update OnrampSelectPaymentCallback.kt
26148cb
@github-actions

github-actions Bot commented Mar 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Diffuse output:

OLD: identity-example-release-base.apk (signature: V1, V2)
NEW: identity-example-release-pr.apk (signature: V1, V2)

          │          compressed          │         uncompressed         
          ├───────────┬───────────┬──────┼───────────┬───────────┬──────
 APK      │ old       │ new       │ diff │ old       │ new       │ diff 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
      dex │   2.2 MiB │   2.2 MiB │  0 B │   4.6 MiB │   4.6 MiB │  0 B 
     arsc │   1.2 MiB │   1.2 MiB │  0 B │   1.2 MiB │   1.2 MiB │  0 B 
 manifest │   2.3 KiB │   2.3 KiB │  0 B │     8 KiB │     8 KiB │  0 B 
      res │   298 KiB │   298 KiB │  0 B │ 447.6 KiB │ 447.6 KiB │  0 B 
   native │  18.6 MiB │  18.6 MiB │  0 B │  18.6 MiB │  18.6 MiB │  0 B 
    asset │   7.9 KiB │   7.9 KiB │  0 B │   7.7 KiB │   7.7 KiB │  0 B 
    other │ 110.1 KiB │ 110.1 KiB │ -1 B │ 235.1 KiB │ 235.1 KiB │  0 B 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
    total │  22.4 MiB │  22.4 MiB │ -1 B │    25 MiB │    25 MiB │  0 B 

 DEX     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
   files │     1 │     1 │ 0         
 strings │ 22058 │ 22058 │ 0 (+0 -0) 
   types │  6589 │  6589 │ 0 (+0 -0) 
 classes │  5318 │  5318 │ 0 (+0 -0) 
 methods │ 32267 │ 32267 │ 0 (+0 -0) 
  fields │ 18595 │ 18595 │ 0 (+0 -0) 

 ARSC    │ old  │ new  │ diff 
─────────┼──────┼──────┼──────
 configs │  158 │  158 │  0   
 entries │ 3979 │ 3979 │  0
APK 
   compressed    │   uncompressed   │                                           
──────────┬──────┼───────────┬──────┤                                           
 size     │ diff │ size      │ diff │ path                                      
──────────┼──────┼───────────┼──────┼───────────────────────────────────────────
  1.2 KiB │ -3 B │   1.2 KiB │  0 B │ ∆ META-INF/CERT.RSA                       
 29.3 KiB │ +2 B │    65 KiB │  0 B │ ∆ META-INF/CERT.SF                        
    272 B │ +1 B │     120 B │  0 B │ ∆ META-INF/version-control-info.textproto 
 25.8 KiB │ -1 B │  64.9 KiB │  0 B │ ∆ META-INF/MANIFEST.MF                    
──────────┼──────┼───────────┼──────┼───────────────────────────────────────────
 56.6 KiB │ -1 B │ 131.2 KiB │  0 B │ (total)

@Twigz Twigz requested review from jeanregisser and silasbw March 24, 2026 21:25
@Twigz Twigz marked this pull request as ready for review March 24, 2026 21:25
@Twigz Twigz requested review from a team as code owners March 24, 2026 21:25
jeanregisser
jeanregisser reviewed Mar 31, 2026
View reviewed changes

@jeanregisser jeanregisser left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice counterpart to the iOS PR 🎉
The handleGooglePayPaymentMethod refactor is clean and neatly avoids the mutable state concern flagged on the iOS side. Main suggestion is adding unit tests for googlePayKycInfo. A few inline notes below.

Comment thread ...o-onramp/src/main/java/com/stripe/android/crypto/onramp/model/OnrampSelectPaymentCallback.kt Outdated
Comment thread crypto-onramp/src/main/java/com/stripe/android/crypto/onramp/OnrampInteractor.kt Outdated
Comment thread crypto-onramp/src/main/java/com/stripe/android/crypto/onramp/OnrampInteractor.kt Outdated
Comment thread crypto-onramp/src/main/java/com/stripe/android/crypto/onramp/OnrampInteractor.kt Outdated
Comment thread crypto-onramp/src/main/java/com/stripe/android/crypto/onramp/OnrampInteractor.kt Outdated
@Twigz Twigz requested a review from jeanregisser March 31, 2026 21:31
jeanregisser
jeanregisser reviewed Apr 2, 2026
View reviewed changes

@jeanregisser jeanregisser left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 🙌 We just need to agree on the rule for splitting name into firstName / lastName.

Comment thread crypto-onramp/src/test/java/com/stripe/android/crypto/onramp/PaymentMethodKtxTest.kt Outdated
@Twigz Twigz requested a review from jeanregisser April 6, 2026 15:29
jeanregisser
jeanregisser approved these changes Apr 6, 2026
View reviewed changes

@jeanregisser jeanregisser left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@Twigz Twigz merged commit 7856418 into master Apr 6, 2026
19 checks passed
@Twigz Twigz deleted the twigz/google-pay-kyc-fill branch April 6, 2026 19:19
@mliberatore mliberatore mentioned this pull request Apr 14, 2026
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeanregisser jeanregisser jeanregisser approved these changes

@silasbw silasbw Awaiting requested review from silasbw

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Twigz @jeanregisser