-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add limit tp (can't make limit sl as far as I know) #46
Comments
I will add this |
Have a look, it now possible |
Nice feature, it effects the fees very much as well. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have noticed the current way tp and sl orders are placed are executed by a market order. This has caused a lot of bad fills that are not where the designated tp or sl was wanted. Adding the option to execute these as limit orders will make sure the tp and sl is executed at the desired price no matter what. With the way limit orders and bybit tp and sl system works, we would not be able to place a limit order for the sl but we can for the tp. This means we would continue using the designated system for sl but for a tp we would need to actually create a limit order in the order book of the pair at the desired percent. I don't think there is any possible way to make the sl a limit order, but there is with certainty a way to make the tp a limit order and this will greatly improve profit and testing.
The text was updated successfully, but these errors were encountered: