Skip to content

Conversation

mchoun
Copy link

@mchoun mchoun commented Aug 1, 2025

This PR is to add an example for One Time Payment Card Fields as well as Save Payment Method Card Fields integrations.

@mchoun mchoun changed the title feat: add Card Fields Example feat: add Card Fields example Aug 1, 2025
@mchoun mchoun force-pushed the feature/add-card-fields-example branch from 2b5f38d to 04def78 Compare August 7, 2025 20:19
// });

// NOTE: Eligibility does not currently return card
// if (paymentMethods.isEligible("card")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we enable this eligibility check?

Suggested change
// if (paymentMethods.isEligible("card")) {
if (paymentMethods.isEligible("card")) {

@mchoun mchoun marked this pull request as ready for review September 25, 2025 18:00
@mchoun mchoun requested a review from a team as a code owner September 25, 2025 18:00
Comment on lines +17 to +19
| Sample Integration | Description |
| ----------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| [Recommended](src/recommended/index.html) | Start with this recommended sample integration. It displays all buttons supported by the v6 SDK and includes eligibility logic. It uses the "auto" presentation mode which attempts to launch a popup window and then automatically falls back to the modal presentation mode when the browser does not support popups. |
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this need to be updated?

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.

3 participants