Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

Issue when call getMyTrades() #157

Closed
truonghatsts opened this issue Jul 20, 2018 · 7 comments
Closed

Issue when call getMyTrades() #157

truonghatsts opened this issue Jul 20, 2018 · 7 comments

Comments

@truonghatsts
Copy link

When I call this BinanceApiRestClientImpl.getMyTrades(), it thrown below exception:

com.binance.api.client.exception.BinanceApiException: com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "symbol" (class com.binance.api.client.domain.account.Trade), not marked as ignorable (11 known properties: "commissionAsset", "qty", "isBestMatch", "price", "isMaker", "commission", "time", "tradeId", "orderId", "id", "isBuyer"])
 at [Source: (okhttp3.ResponseBody$BomAwareReader); line: 1, column: 13] (through reference chain: java.util.ArrayList[0]->com.binance.api.client.domain.account.Trade["symbol"])
        at com.binance.api.client.impl.BinanceApiServiceGenerator.executeSync(BinanceApiServiceGenerator.java:60)
        at com.binance.api.client.impl.BinanceApiRestClientImpl.getMyTrades(BinanceApiRestClientImpl.java:183)
        at com.binance.api.client.impl.BinanceApiRestClientImpl.getMyTrades(BinanceApiRestClientImpl.java:193)

@joaopsilva : I believe it's because of the new api change.

@DanielSBing
Copy link

I get same error

@pahgo
Copy link

pahgo commented Jul 20, 2018

The issue seems to be when there is content in the response as it contains a field called symbol that the Trade object doesn't have. I created the PR #159

@sharatsahni1
Copy link

I get the same error .. any timelines as to when would this be resolved ?

@joaopsilva
Copy link
Member

It's fixed now, thanks for the contrib @pahgo .

@sharatsahni1
Copy link

i am still getting the same error

@joaopsilva
Copy link
Member

@sharatsahni1 please update the repo.

@sharatsahni1
Copy link

It was some configuration issue at my end. Its working fine now - Thanks

nilswx pushed a commit to nilswx/binance-java-api that referenced this issue Dec 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants