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/upgrade pool #46

Open
wants to merge 67 commits into
base: feat/prod
Choose a base branch
from
Open

Feat/upgrade pool #46

wants to merge 67 commits into from

Conversation

MuesliW
Copy link
Contributor

@MuesliW MuesliW commented Sep 15, 2023

No description provided.

MuesliW and others added 30 commits August 16, 2023 14:13
pull upstream change - better env variable management
1. ATokenMagpieStaker that inherit AToken
- forbid borrow/flashloan
- stake/unstake LP upon mint/burn
- allow admin to claim rewards and send it over to EmissionAdminAndDirectTransferStrategy

2. EmissionAdminAndDirectTransferStrategy
- keep rewards token for emission
- act as the transferStrategy too
- direct rewards to emissionManager
…P withdraw, but only direct underlying redeem
userCollateralInBaseCurrency check is safe to remove, since the debt asset would be non-zero in base currency and collateralNeededInBaseCurrency
would be non-zero too, thus userCollateralInBaseCurrency has to be bigger than collateralNeededInBaseCurrency is a sufficient check (L263)
userCollateralInBaseCurrency check is safe to remove, since the debt asset would be non-zero in base currency and collateralNeededInBaseCurrency
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.

2 participants