-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add comments on lost fees due to bad debt or market forced removals #359
Conversation
Co-authored-by: Merlin Egalite <44097430+MerlinEgalite@users.noreply.github.com> Signed-off-by: Jean-Grimal <83286814+Jean-Grimal@users.noreply.github.com>
Co-authored-by: Romain Milon <rmilon@gmail.com> Signed-off-by: Jean-Grimal <83286814+Jean-Grimal@users.noreply.github.com>
ce04113
Co-authored-by: Romain Milon <rmilon@gmail.com> Signed-off-by: Jean-Grimal <83286814+Jean-Grimal@users.noreply.github.com>
299229e
Co-authored-by: Romain Milon <rmilon@gmail.com> Signed-off-by: Jean-Grimal <83286814+Jean-Grimal@users.noreply.github.com>
…bs/morpho-blue-metamorpho into feat/add-comments-on-lost-fees
99651fd
@@ -145,6 +147,7 @@ interface IMetaMorphoBase { | |||
/// @notice Warning: Anyone can supply on behalf of the vault so the call to `updateWithdrawQueue` that expects a | |||
/// market to be empty can be griefed by a front-run. To circumvent this, the allocator can simply bundle a | |||
/// reallocation that withdraws max from this market with a call to `updateWithdrawQueue`. | |||
/// @dev Warning: Removing a market with supply will decrease the fee accrued until the next deposit/withdrawal. |
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.
If the amount removed is bigger than the total amount of interest accrued by the other markets, the fee to accrue will be zero. It's worth mentioning it in the comment or anyway be more specific.
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.
can do it yes even though I think this is the general case
Replaces #337
Fixes https://cantina.xyz/ai/8409a0ce-6c21-4cc9-8ef2-bd77ce7425af/findings/5a81700d-ff95-422a-be37-e24f7e4d50a9