search

coinbase / coinbase-pro-node

DEPRECATED — The official Node.js library for Coinbase Pro
Apache License 2.0
844 stars 316 forks source link

#370 Add getPaymentMethods to the AuthenticatedClient class types def… #371

Closed theRealScoobaSteve closed 4 years ago

theRealScoobaSteve commented 5 years ago

Fixes #370, Did not see a point in adding a test for this one.

jonlorusso commented 5 years ago

@theRealScoobaSteve I believe getPaymentMethods needs to return an array of PaymentMethods rather than a CoinbaseAccounts?

https://github.com/coinbase/coinbase-pro-node/pull/375

theRealScoobaSteve commented 5 years ago

@theRealScoobaSteve I believe getPaymentMethods needs to return an array of PaymentMethods rather than a CoinbaseAccounts?

375

I would say you are correct sir, will fix

drewrothstein commented 4 years ago

Hi, we are closing out PRs + Issues as this project is being archived.