Skip to content
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

Open
gtmoon32 opened this issue Apr 9, 2023 · 3 comments
Open

Add limit tp (can't make limit sl as far as I know) #46

gtmoon32 opened this issue Apr 9, 2023 · 3 comments

Comments

@gtmoon32
Copy link

gtmoon32 commented Apr 9, 2023

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.

@Bennch
Copy link
Contributor

Bennch commented Apr 13, 2023

I will add this

@Bennch
Copy link
Contributor

Bennch commented Apr 21, 2023

Have a look, it now possible

@nvisible88
Copy link

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants