bitfinexcom / bitfinex-api-go

BITFINEX Go trading API - Bitcoin, Litecoin, and Ether exchange
https://www.bitfinex.com/
MIT License
310 stars 220 forks source link

New order endpoints #191

Closed JacobPlaster closed 5 years ago

JacobPlaster commented 5 years ago

Description:

This pull request adds helper functions to interact with the newly added endpoints to the v2 rest Bitfinex API. Also some older funding endpoints seemed to be missing so I have added those in as well.

Breaking changes:

None

New Functions:

Fixes:

None

PR status:

andreygrehov commented 5 years ago

Calling Rest.Wallet.DepositAddress results to a

(*bitfinex.Notification)(0xc0000b81e0)({
 MTS: (int64) 1574137008828,
 Type: (string) (len=7) "acc_dep",
 MessageID: (int64) 0,
 NotifyInfo: (interface {}) <nil>,
 Code: (int64) 0,
 Status: (string) (len=7) "SUCCESS",
 Text: (string) (len=7) "success"
})

Is this a bug or am I doing something wrong?

JacobPlaster commented 5 years ago

Hey @andreygrehov, the response is correct. The payload of the notification NotifyInfo contains the data that you are looking for.

andreygrehov commented 5 years ago

@JacobPlaster but NotifyInfo is nil.

andreygrehov commented 5 years ago

@JacobPlaster up

JacobPlaster commented 5 years ago

Fixed in https://github.com/bitfinexcom/bitfinex-api-go/pull/203