-
Notifications
You must be signed in to change notification settings - Fork 648
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
LinkAccount payment methods list #9616
Conversation
2152424
to
2a639fe
Compare
2a639fe
to
9bb88ff
Compare
Diffuse output:
APK
DEX
|
@@ -1435,6 +1437,27 @@ class StripeApiRepository @JvmOverloads internal constructor( | |||
) | |||
} | |||
|
|||
override suspend fun listPaymentDetails( | |||
clientSecret: String, | |||
paymentMethodTypes: Set<String>, |
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 parameter needed? It looks like it's only set to emptySet and is otherwise unused (I could be missing something though)
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.
It is. I was supposed to add it to the other interfaces as well. Just pushed a fix.
Summary
update link account manager to allow consumer payment methods to be fetched
Motivation
JIRA
Testing
Changelog