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.

Sign up for GitHub

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

Jump to bottom

Add limit for total number of active coins #147

Closed
redhelix opened this issue May 4, 2021 · 3 comments · Fixed by #311
Closed

Add limit for total number of active coins #147

redhelix opened this issue May 4, 2021 · 3 comments · Fixed by #311
Labels
enhancement New feature or request

Comments

@redhelix
Copy link

redhelix commented May 4, 2021 •
edited
Loading

Is your proposal related to a problem?

When looking for the best deals, I like to add many different coin options. When many coins reach possible trade conditions, there is a risk that the bot will draw down the available balance very low or end up with a lot of trades in play.

Describe the solution you'd like

Add a parameter to limit total number of active deals

Describe alternatives you've considered

reducing the number of coins to monitor in the list - means foregoing potential profits

Additional context
@redhelix redhelix added the enhancement New feature or request label May 4, 2021
@chrisleekr
Copy link
Owner

chrisleekr commented May 4, 2021 •
edited
Loading

Hmm, so like if I limit to 5 deals and there are 5 open orders, you don't want more buy?

@redhelix
Copy link
Author

redhelix commented May 5, 2021

Correct - Could limit by bases - 5 USDT, 5 BTC etc. Wait till coins cycle out could also sort by closest to buy price

@chrisleekr
Copy link
Owner

chrisleekr commented May 6, 2021 •
edited
Loading

I actually found this feature needed when all coins were going down and placing a lot of orders for many coins at the same time.

I will add it to the TODO list.

@chrisleekr chrisleekr mentioned this issue May 17, 2021
Closed
This was referenced Sep 8, 2021
Closed
Merged
@chrisleekr chrisleekr linked a pull request Sep 12, 2021 that will close this issue
feat: limit for buy open orders/open trades #311
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: limit for buy open orders/open trades chrisleekr/binance-trading-bot
2 participants
@redhelix @chrisleekr