CryptoExchange.Net icon indicating copy to clipboard operation
CryptoExchange.Net copied to clipboard

Miss (Parameter 'accountId')) on common PlaceOrderAsync for Huobi

Open heavymanto opened this issue 1 year ago • 0 comments

Describe the bug

Using Houbi PlaceOrderAsync by ISpotClient it raise Exception: One or more errors occurred. (accountId required for Huobi PlaceOrderAsync (Parameter 'accountId'))

Using Houbi client with SpotClient work fine

Unfortunately Huobi also wants accountId as a parameter which obviously is not present as a generic parameter. As an alternative to adding it to the list of parameters since, in a generic wrap, a specific parameter is out of place, you could prepare a generic parameter such as List key/value. This way you can smartly manage the specifics of some Exchanges, without changing the code in the future. What do you say? I would also use this trick on the other methods of the common interface, with default to null, in this way the backward compatibility of the methods that call the interface is guaranteed.

Thank you

heavymanto avatar May 23 '24 08:05 heavymanto