Skip to content

Conversation

josh-burton
Copy link

Fixes #305

Describe the changes proposed

Adds a new existingPaymentMethodRequired parameter to the userCanPay method, so the developer can determine if Apple/Google pay is available without requiring cards to be added

Screenshots / Videos

If useful, include any screeshots or screencasts to better explain the change.

Additional context

Add any other context about the change here.

…out requiring cards to be added

Adds a new existingPaymentMethodRequired parameter to the userCanPay method, so the developer can determine if Apple/Google pay is available without requiring cards to be added
@JlUgia
Copy link
Member

JlUgia commented Jul 29, 2025

Hi @josh-burton, thank you for your contributions.
My preference would be to default to false on this new prop, in line with how both enabled providers operate today.
Also, how about we use a more straightforward prop name on the Flutter API surface? Both Apple and Google Pay have slight variations on how the API works, so I feel that something like existingPaymentMethodAvailable or similar would be more accurate, since we are looking to extract whether there's at least one payment method available for the configuration provided.

Thank you again.

@josh-burton
Copy link
Author

Hi @josh-burton, thank you for your contributions. My preference would be to default to false on this new prop, in line with how both enabled providers operate today. Also, how about we use a more straightforward prop name on the Flutter API surface? Both Apple and Google Pay have slight variations on how the API works, so I feel that something like existingPaymentMethodAvailable or similar would be more accurate, since we are looking to extract whether there's at least one payment method available for the configuration provided.

Thank you again.

Thank you for the review. I have made those changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support userCanPay when no cards added
2 participants