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

BrokerID #41

Closed overstartup closed 11 months ago

overstartup commented 1 year ago

Could you please add set to brokerid property? I know maybe it is different from your business plan, but I need to set it for my software. public string BrokerId { get; }

I appreciate your effort.

burakoner commented 1 year ago

It doesnt effect your orders. If you have a broker account you'll need to create a new class that inherits ClientOptions

overstartup commented 1 year ago

thanks. i will give it try

overstartup commented 1 year ago

Do you mean like this:

 public class clientoption : OKXRestApiClientOptions
    {
        public string ApiKey { get; set; }
        public string SecretKey { get; set; }
        public string Passphrase { get; set; }
        public bool DemoTradingService { get; set; }
        public string BrokerId { get; set; }

    }
    `    api = new OKXRestApiClient(new clientoption { DemoTradingService = configurationOptions.DemoTradingService, LogLevel = LogLevel.Trace, BrokerId = "test" });`

but it shows 2 brokerID. image

overstartup commented 11 months ago

@burakoner so there is no solution?

burakoner commented 11 months ago

You need to solve this issue yourself. I'm developing OKX wrapper since Jan 26, 2020. I spent my own money and valuable time without getting any profit. If you have your own broker id or dont want to use mine, this is your problem not related with repository

overstartup commented 11 months ago

You need to solve this issue yourself. I'm developing OKX wrapper since Jan 26, 2020. I spent my own money and valuable time without getting any profit. If you have your own broker id or dont want to use mine, this is your problem not related with repository

@burakoner i appreciate your effort.i will sponsor you when my startup gets active.