-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fixed Lens withdrawable calculation #1253
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.
A quick win test is to check if you withdraw a bit more than withdrawable, It should revert
Morpho-aave-v2 gas impacts (eth-mainnet)
🧾 Summary
Full diff report 👇
|
Morpho-aave-v2 gas impacts (polygon-mainnet)
🧾 Summary
Full diff report 👇
|
Morpho-aave-v2 gas impacts (avalanche-mainnet)
🧾 Summary
Full diff report 👇
|
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.
Just one question but overall agree with the change
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.
Great work ! Some changes to do. Also maybe you can decompose the code into smaller part, for example with the _testShouldRepayMarketP2PAndFromPool
test
2d76772
to
ab30517
Compare
e0d19a5
to
54a9ed9
Compare
Morpho-compound gas impacts (eth-mainnet)
🧾 Summary
Full diff report 👇
|
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.
LGTM
54a9ed9
to
4e957d4
Compare
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.
LGTM
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.
LGTM
Pull Request
Issue(s) fixed
The test added was failing before