search

irufus / gdax-java

Java based wrapper for Coinbase Pro (Formerly known as GDAX API and Coinbase Exchange API)
MIT License
177 stars 131 forks source link

Withdrawals to coinbase and crypto address do not work #24

Closed duanemay closed 6 years ago

duanemay commented 6 years ago

For coinbase the URI should be /withdrawals/coinbase-account

On both of these a different RequestType is needed

I have a fix on my fork https://github.com/duanemay/gdax-java/commit/02e41b4b50478d5c9660c2e6dc3474cd23a9ec35

robevansuk commented 6 years ago

looks good @irufus - endpoint needs an update. Merge when ready

irufus commented 6 years ago

@robevansuk Can't merge on that commit as it drags in a bunch of unwanted changes. A few simple updates to the code base should solve the problem.

irufus commented 6 years ago

This should addressed a majority of the issues. We can't test certain payments in sandbox but we at least have addressed the endpoints.