search

burakoner / OKX.Api

Up-to-date, most-complete, well-organized, well-documented, easy-to-use, multi-task and multi-thread compatible OKX Cryptocurrency Exchange Rest and Websocket Api Wrapper
MIT License
39 stars 15 forks source link

after upgrade, GetAccountBalanceAsync doesnt work #35

Closed kayahasa closed 1 year ago

kayahasa commented 1 year ago

TradingAccount.GetAccountBalanceAsync always returns {: No credentials provided for private endpoint } even i have entered credentials. Other endpoints working.

Also,

SubscribeToCandlesticksAsync still doesnt work for multiple subscription at once. Always only BTC-USD-SWAP candle is coming. Others are not coming.

But GetAccountBalanceAsync is mor urgent :(

kayahasa commented 1 year ago

GetAccountPositionsAsync also same

kayahasa commented 1 year ago

well i think all private endpoints are not working. 

        var client = new OKXRestApiClient();
        client.SetApiCredentials(credentials.ApiKey, credentials.SecretKey, credentials.PassPhrase);

        Is this incorrect? It was working before big update.
kayahasa commented 1 year ago 
/// <summary>
/// Sets API Credentials
/// </summary>
/// <param name="credentials">OkxApiCredentials Object</param>
public void SetApiCredentials(OkxApiCredentials credentials)
{
    FundingAccount.SetApiCredentials(credentials);
    OrderBookTrading.SetApiCredentials(credentials);
    BlockTrading.SetApiCredentials(credentials);
    SpreadTrading.SetApiCredentials(credentials);
    PublicData.SetApiCredentials(credentials);
    TradingStatistics.SetApiCredentials(credentials);
    SubAccount.SetApiCredentials(credentials);
    FundingAccount.SetApiCredentials(credentials);
    Status.SetApiCredentials(credentials);
}
Here, your forget to add TradingAccount. That is the reason i think.
kayahasa commented 1 year ago

https://github.com/burakoner/OKX.Api/pull/36/files

I have created PR, check please

burakoner commented 1 year ago

It's my mistake. Thank you for PR. New version will be released in minutes