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

Fix/sell order hanging #2124

Closed
wants to merge 6 commits into from
Closed

Fix/sell order hanging #2124

wants to merge 6 commits into from

Conversation

xmatthias
Copy link
Member

@xmatthias xmatthias commented Aug 12, 2019

Summary

Market orders normally close immediately, so we should handle this accordingly.
This should prevent issues as seen in #2116 (however not once an order is stuck obviously).

Closes #2116
Closes #1053

Quick changelog

  • dry-run returning closed market-orders immediately
  • tests for new behaviour

Copy link
Member

@hroff-1902 hroff-1902 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems that this LGTM 👍

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 96.745% when pulling 9f26c4e on fix/sell_order_hanging into e0e5011 on develop.

@xmatthias
Copy link
Member Author

Github does not seem to have a good week ...
This PR is merged (3af5691) using the UI, even ... but github does not see it, therefore closing this (but it's merged!).

@xmatthias xmatthias closed this Aug 15, 2019
xmatthias added a commit that referenced this pull request Aug 15, 2019
@xmatthias xmatthias deleted the fix/sell_order_hanging branch August 16, 2019 08:18
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

Successfully merging this pull request may close these issues.

Orders not closing in huobi Trades DB updates on next loop around pairs after close
3 participants