Skip to content
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

Sundry fixes #9

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Sundry fixes #9

wants to merge 9 commits into from

Conversation

ekkis
Copy link
Collaborator

@ekkis ekkis commented May 2, 2022

  • added currency support
  • enhanced country list, as per official documentation
  • now does proper validation on partner ID and credentials
  • fixed tests (but they need proper credentials)
  • enhancements to constructor
  • ABC: card creation must now be passed a country code instead of a region. currency no longer passed (derived internally)
  • updated documentation

ekkis added 9 commits May 1, 2022 18:17
- enhanced country list, as per official documentation
- now does proper validation on partner ID and credentials
- fixed tests (but they need proper credentials)
- enhancements to constructor
- ABC: card creation must now be passed a country code instead of a region.  currency no longer passed (derived internally)
- updated documentation
the default template must be sourced from the project itself, not the client's config directory as the template is maintained by this project, not the client
'sandbox' is not a valid NODE_ENV value
- replaced deprecated 'request' with standard 'node-fetch'
- implemented promises, maintaining support for callbacks
- APB: public functions no longer return debugging information as this is returned in the promise or callback
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