-
Notifications
You must be signed in to change notification settings - Fork 629
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
Pre-Populate the form #4124
Pre-Populate the form #4124
Conversation
…ate form field values.
…ate form field values.
…to populate form field values." This reverts commit c9489e8.
Diffuse output: |
/** | ||
* The billing information for the user. | ||
* | ||
* If set, PaymentSheet will pre-populate the form fiels with the values provided. |
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.
* If set, PaymentSheet will pre-populate the form fiels with the values provided. | |
* If set, PaymentSheet will pre-populate the form fields with the values provided. |
city = "San Francisco", | ||
state = "CA", | ||
postalCode = "94111", | ||
country = "DE", |
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.
Is "DE" with a US address on purpose?
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.
yeah, United States is the default.
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.
I switched it to a Dublin address
) : Parcelable | ||
|
||
@Parcelize | ||
data class Address( |
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.
Is this being redefined just because of the dependency direction? Will it change to be the PaymentSheet.Address
?
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.
yeah
payments-core/src/main/java/com/stripe/android/paymentsheet/BaseAddPaymentMethodFragment.kt
Outdated
Show resolved
Hide resolved
FYI this is the iOS implementation https://github.com/stripe-ios/stripe-ios/pull/335 |
Summary
Pre-Populate the form with billing details.
Motivation
Testing
Screenshots