Skip to content
This repository has been archived by the owner on Jan 20, 2020. It is now read-only.

Withdraw to a crypto address. #90

Merged
merged 2 commits into from
Sep 6, 2017
Merged

Conversation

passabilities
Copy link
Contributor

No description provided.

@fb55
Copy link
Contributor

fb55 commented Aug 7, 2017

Could you add a test and document it in the README?

README.md Outdated
const withdrawAddressParams = {
'amount': 10.00,
'currency': 'BTC',
'crypto_address': '1CQ4V3zij8v8BxHEEGQnTshJfW2ZRTEXes'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please replace this with something that's not a valid address.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok i changed it

@passabilities passabilities force-pushed the withdraw-crypto branch 2 times, most recently from 07c1d16 to adad226 Compare August 16, 2017 22:42
@fb55
Copy link
Contributor

fb55 commented Aug 25, 2017

This looks good, only needs another rebase

@fb55 fb55 merged commit 6fd3d51 into coinbase:master Sep 6, 2017
@fb55
Copy link
Contributor

fb55 commented Sep 6, 2017

Sorry, didn't see that this was rebased. Thanks a lot!

@passabilities passabilities deleted the withdraw-crypto branch October 6, 2017 23:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants