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

payment-circle: extend sendPayment functionality to allow sendding Circle->BankWire payments #21

Merged
merged 6 commits into from
Feb 3, 2022

Conversation

marcelosalloum
Copy link
Contributor

PR Checklist

PR Structure

  • This PR has reasonably narrow scope (if not, break it down into smaller PRs).
  • This PR avoids mixing refactoring changes with feature changes (split into two PRs
    otherwise).
  • This PR's title starts with name of package that is most changed in the PR, ex.
    paymentservice.stellar, or all or doc if the changes are broad or impact many
    packages.

Thoroughness

  • This PR adds tests for the most critical parts of the new functionality or fixes.

What

Extend sendPayment functionality to allow sendding Circle->BankWire payments.

Why

This is a feature that one of our potential adopters needs and it's likely most partners will need it too.

@marcelosalloum marcelosalloum self-assigned this Feb 2, 2022
@marcelosalloum marcelosalloum requested review from JakeUrban and lijamie98 and removed request for JakeUrban February 2, 2022 22:31
@marcelosalloum marcelosalloum marked this pull request as ready for review February 2, 2022 22:31
@marcelosalloum
Copy link
Contributor Author

I'm merging this PR to unblock Jamie. I can address further feedback in the next one.

@marcelosalloum marcelosalloum merged commit 8878889 into main Feb 3, 2022
@marcelosalloum marcelosalloum deleted the payments-send-payout branch February 3, 2022 16:51
lijamie98 pushed a commit that referenced this pull request Feb 3, 2022
…ircle->BankWire` payments (#21)

Extend sendPayment functionality to allow sendding `Circle->BankWire` payments.

This is a feature that one of our potential adopters needs and it's likely most partners will need it too.
lijamie98 pushed a commit that referenced this pull request Feb 3, 2022
…ircle->BankWire` payments (#21)

Extend sendPayment functionality to allow sendding `Circle->BankWire` payments.

This is a feature that one of our potential adopters needs and it's likely most partners will need it too.
@reecexlm reecexlm added this to the Payment-Circle milestone Feb 8, 2022
@marcelosalloum marcelosalloum linked an issue Feb 18, 2022 that may be closed by this pull request
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement payments gateway interface for the Circle API
4 participants