search

Kucoin / kucoin-futures-python-sdk

MIT License
7 stars 3 forks source link

market order: size added #26

Closed engblnk closed 7 months ago

engblnk commented 2 years ago

First of all, thanks for this lib and your effort.

I was not able use market order, so I fixed it. :)

codewc commented 2 years ago

First of all, thanks for this lib and your effort.

I was not able use market order, so I fixed it. :)

sorry, I don't think the size parameter is necessary, because the endpoint has many parameters, there are non-essential parameters to be replaced by **kwags, what do you think?

jenoOvchi commented 8 months ago

First of all, thanks for this lib and your effort. I was not able use market order, so I fixed it. :)

sorry, I don't think the size parameter is necessary, because the endpoint has many parameters, there are non-essential parameters to be replaced by **kwags, what do you think?

Hi! This PR is necessary to fix this issue: https://github.com/Kucoin/kucoin-futures-python-sdk/issues/47 Or we need to hardcode this param - without it we can't create an order.