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

Support retry for short trade #887

Open
hieuh25 opened this issue Apr 16, 2024 · 0 comments
Open

Support retry for short trade #887

hieuh25 opened this issue Apr 16, 2024 · 0 comments
Assignees
@hieuh25
Labels
enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus.

Comments

@hieuh25
Copy link
Contributor

hieuh25 commented Apr 16, 2024

Currently short trades can be failed without proper reason with just execution reverted message. Fixing this problem post-mortem proved to be very difficult.
Instead we could have retry logic with some wait time (e.g: 30s) to retry these failed trades a few times before giving up totally
@hieuh25 hieuh25 self-assigned this Apr 16, 2024
@hieuh25 hieuh25 added enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus. labels Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hieuh25 hieuh25

Labels
enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hieuh25