-
Notifications
You must be signed in to change notification settings - Fork 19
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
Bugfix/2103 stop withdrawal if amount too big #2117
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The withdraw lock account seems like a good approach. I guess it'll need a timeout or something so funds don't get stuck there forever?
Absolutely, it's mentionned in the ticket for the withdrawal of erc20 that we want to have a timeout on that. Initial version won't have this, and we'll take care of it later. I prefer to finalize the whole withdrawal before introduce the timeout mecanics. |
@jeremyletang Great - I've put a ticket on https://github.com/orgs/vegaprotocol/projects/24#card-44024089 so we don't forget about that |
7c515a6
to
85ea306
Compare
This has been rebased + integration tests fixed and is therefore ready to merge. |
system-tests failed.
|
Add a LockFund step to be used later which ensure we have enough funds to withdraw.
return an error if cannot or process.
closes #2103