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

feat: partial liquidations and minor improvements #217

Merged
merged 2 commits into from
May 13, 2024

Conversation

lekhovitsky
Copy link
Collaborator

@lekhovitsky lekhovitsky commented May 12, 2024

That's the very commit one would expect to see the night before the audit kicks in.

Here we add the logic from PartialLiquidationBotV3 to the credit facade. This approach has a few important benefits:

  • it doesn't trigger safe pricing in the collateral check
  • it allows partial liquidations during pause
  • it properly handles fee-on-transfer underlying

The PR also contains some general code improvements:

  • CreditConfiguratorV3's constructor is more consistent
  • feeInterest is made immutable in CreditManagerV3, which prevents debt manipulation by controller and improves storage packing
  • some dead code is removed from libraries

That's the very commit one would expect to see the night before the audit kicks in.

Here we add the logic from `PartialLiquidationBotV3` to the credit facade.
This approach has a few important benefits:
* it doesn't trigger safe pricing in the collateral check
* it allows partial liqudiations during pause
* it properly handles fee-on-transfer underlyings
@lekhovitsky lekhovitsky merged commit e3ff71e into next May 13, 2024
3 checks passed
@lekhovitsky lekhovitsky deleted the partial-liquidations branch May 13, 2024 07:14
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.

1 participant