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

Fixed LI-CF PreLiquidation #1

Merged
merged 192 commits into from
Sep 25, 2024
Merged

Fixed LI-CF PreLiquidation #1

merged 192 commits into from
Sep 25, 2024

Conversation

peyha
Copy link
Collaborator

@peyha peyha commented Aug 30, 2024

PR for the Morpho Liquidation Protection Contract

The general idea is that this contract will allow borrowers to be pre-liquidated on some Morpho market for specific pre-liquidation parameters.
A factory can deploy pre-liquidation contracts each implementing pre-liquidation for its own specific parameters and its own specific morpho market.
A borrower can then authorize the pre-liquidation contract on morpho which will allow liquidators to close part of the position when it is insolvent under the pre-liquidation parameters

Copy link
Contributor

@QGarchery QGarchery left a comment

Choose a reason for hiding this comment

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

🔥

src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
src/LiquidationProtection.sol Outdated Show resolved Hide resolved
@MathisGD MathisGD changed the title First working version Fixed LI-CF PreLiquidation Sep 24, 2024
@MathisGD MathisGD assigned MathisGD and peyha and unassigned MathisGD Sep 24, 2024
src/PreLiquidation.sol Outdated Show resolved Hide resolved
src/PreLiquidation.sol Show resolved Hide resolved
src/PreLiquidation.sol Outdated Show resolved Hide resolved
QGarchery
QGarchery previously approved these changes Sep 25, 2024
src/PreLiquidation.sol Outdated Show resolved Hide resolved
src/PreLiquidation.sol Show resolved Hide resolved
src/PreLiquidation.sol Outdated Show resolved Hide resolved
src/PreLiquidation.sol Outdated Show resolved Hide resolved
src/PreLiquidation.sol Outdated Show resolved Hide resolved
QGarchery
QGarchery previously approved these changes Sep 25, 2024
colin-morpho
colin-morpho previously approved these changes Sep 25, 2024
Copy link
Contributor

@colin-morpho colin-morpho left a comment

Choose a reason for hiding this comment

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

Some comments to resolve but LGTM otherwise 🚀

@peyha peyha dismissed stale reviews from colin-morpho and QGarchery via fc5728d September 25, 2024 14:38
@peyha peyha merged commit 9ae4f40 into main Sep 25, 2024
1 check passed
@peyha peyha deleted the feat/licence branch September 25, 2024 14:54
@colin-morpho
Copy link
Contributor

Should we delete branches after they've been merged?

@QGarchery
Copy link
Contributor

QGarchery commented Sep 25, 2024

Should we delete branches after they've been merged?

Yes because then on github the PRs on top have their base branch changed automatically, which prevents some mistakes when merging multiple PRs. So for example #19 base branch is now main (and we wouldn't want it to be feat/licence):
Screenshot from 2024-09-25 18-30-23

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.

4 participants