Skip to content

Add new card fields component #84

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

Merged
merged 3 commits into from
Oct 5, 2023
Merged

Add new card fields component #84

merged 3 commits into from
Oct 5, 2023

Conversation

gregjopa
Copy link
Contributor

@gregjopa gregjopa commented Oct 3, 2023

This PR adds the new Checkout purchase example for card-fields. The example will live under the /advanced-integration/beta folder structure for now.

Notable changes:

  1. Remove ejs since we want to make it as easy as possible to teach. Similar to how the Standard Integration example for Buttons does not use any templating language.
  2. Use the client/server folder structure for code organization.
  3. Use default linter and prettier settings (except for html because prettier is too opinionated with closing tags)

The vault example can be added in a future PR.

Steps to test

  1. Checkout this upstream branch new-card-fields
  2. cd advanced-integration/beta
  3. Add a .env file and use the format in .env.example
  4. npm install
  5. npm start
  6. Use a test sandbox credit card from https://developer.paypal.com/api/rest/sandbox/card-testing/#link-creditcardgeneratorfortesting

@gregjopa gregjopa force-pushed the new-card-fields branch 3 times, most recently from bff0f40 to 6ee5ee2 Compare October 4, 2023 21:16
@gregjopa gregjopa marked this pull request as ready for review October 4, 2023 21:19
@gregjopa gregjopa requested a review from wsbrunson October 4, 2023 21:25
@gregjopa gregjopa merged commit 5c89c3c into main Oct 5, 2023
@gregjopa gregjopa deleted the new-card-fields branch October 5, 2023 21:39
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.

1 participant