-
-
Notifications
You must be signed in to change notification settings - Fork 7.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Kraken: Support for createMarketOrderWithCost using "viqc" flag #22521
Comments
Hello @mjwvb, thanks for bringing our attention to this; we will implement it as soon as possible! |
@mjwvb Actually I'm testing it and this is only supported for market-buy orders
with market sell, throws this error |
hum, market buy also throws an error 🤔
|
@mjwvb apparently only works with
|
@carlosmiei That's very unfortunate. Apparently there's also another version of the docs which says: Interesting how there's two types of api docs? It seems that /api/ is officially linked, and that /rest/ can only be found on google. Oh well... Still weird how it not works for the USDT pair though... Anyway thanks for checking and putting in all the effort! I should have checked the API a bit more, but was a little bit on a rush. So sorry for that :) |
@mjwvb Yeah the docs and the API are very confusing in this regard. Nonetheless, we will implement the |
@carlosmiei Alright! According to the docs you should also exclude margin orders, but cannot confirm right now. Also now that you're into the oflags, I saw in the code that when using postOnly on limit orders, the oflags params is always forced set to "post" instead of appending any existing oflags. It means you cannot provide your own oflags in combination with the native postOnly. See: Line 1789 in 9ab5996
|
@mjwvb Yes you're right, I will update it to take that into consideration. |
Thanks! I'll be testing it out in the near future. By the way, I think USDT markets actually are supported but the minimum order size of LTC/USDT is 10 USDT, and you provided 5 USDT. At least that's what the kraken UI is telling me: "Cannot be smaller than 10 USDT" |
@mjwvb omg you're right 😅, that was the problem, thanks once again! will be updated shortly
|
@carlosmiei haha, Kraken has ridiculous minimum sizes though... |
@mjwvb we just released a new version with the fixes, can you please update your ccxt version and try again? |
@mjwvb turns out we have a small issue with the pipeline so we didn't release yet |
Am I right the release is still pending? Npm can't find 4.3.26 |
@mjwvb Yes the pipeline is not stable yet, unfortunately, but we're trying to fix it right now |
@mjwvb We just released a nee version, 4.3.27, please let us know if anything |
@carlosmiei using 4.3.27 now and it seems to be working perfectly. Thank you once again! |
@mjwvb No problem! |
Preliminary Checks
Is your feature request related to a problem? Please describe
In CCXT there's currently no support for createMarketOrderWithCost for Kraken, although there's support for it in the Kraken API.
Describe the solution you'd like
According to the API docs there seems to be support for a "olflag" named "viqc": "order volume expressed in quote currency. This is supported only for market orders."
See the api docs for addOrder: https://docs.kraken.com/rest/#tag/Spot-Trading/operation/addOrder
I could be wrong, but it sounds like that's what we need to implement market orders with cost / quote size, so we could implement the following functions:
createMarketOrderWithCost() / createMarketBuyOrderWithCost / createMarketSellOrderWithCost
Describe alternatives you've considered
No response
The text was updated successfully, but these errors were encountered: