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
Add country code to PaymentIntent
and SetupIntent
#5472
Conversation
Diffuse output:
APK
DEX
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is the country_code coming from? I don't see it in the public docs.
@jameswoo-stripe You’re right — it’s not a public field, but something we fetch via Since it’s not a field in the public API of a payment/setup intent, should I make this an internal field? |
Yea, that makes more sense to be internal. Also, let's add some tests to test the parsing using fixtures. |
payments-core/src/main/java/com/stripe/android/model/StripeIntent.kt
Outdated
Show resolved
Hide resolved
Having the country code as a library-internal property in StripeIntent isn't easy right now. We're adding it as separate properties in SetupIntent and PaymentIntent for now, before we think about ways to refactor StripeIntent.
We chatted about this on Zoom. TL;DR: Adding |
StripeIntent
PaymentIntent
and SetupIntent
Summary
This pull request adds the country code to
StripeIntent
.Motivation
This will be needed for the name collection in Link.
Testing
Screenshots
🙅
Changelog
Nothing to add.