You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is a ledger method in CoinbasePro interface and it has a Integer startingOrderId parameter. I think it should be a String because in some case orderId is greater then maxiumum value of Integer. Additionally, CoinbasePro documentation says that this parameter should be a String -> https://docs.cloud.coinbase.com/exchange/reference/exchangerestapi_getaccountledger
@Path("accounts/{account_id}/ledger")
@Consumes(MediaType.APPLICATION_JSON)
List<Map<?, ?>> ledger(
@HeaderParam("CB-ACCESS-KEY") String apiKey,
@HeaderParam("CB-ACCESS-SIGN") ParamsDigest signer,
@HeaderParam("CB-ACCESS-TIMESTAMP") long timestamp,
@HeaderParam("CB-ACCESS-PASSPHRASE") String passphrase,
@PathParam("account_id") String accountId,
@QueryParam("after") Integer startingOrderId) //should be a String
throws CoinbaseProException, IOException;
What do you think about this change? Please consider it.
The text was updated successfully, but these errors were encountered:
There is a
ledger
method inCoinbasePro
interface and it has aInteger startingOrderId
parameter. I think it should be aString
because in some case orderId is greater then maxiumum value ofInteger
. Additionally, CoinbasePro documentation says that this parameter should be aString
-> https://docs.cloud.coinbase.com/exchange/reference/exchangerestapi_getaccountledgerWhat do you think about this change? Please consider it.
The text was updated successfully, but these errors were encountered: