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.

Sign up for GitHub

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

Liquidation protocol fee #71

Closed
wants to merge 22 commits into from
Closed

Liquidation protocol fee #71

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
ccd0198
add reserve fields + update pack and unpack
Jan 5, 2022
7fed6b3
update cli
Jan 6, 2022
d3d9746
comment
Jan 6, 2022
cdb807f
add protocol fee to liquidate
Jan 7, 2022
592383f
formatting
Jan 10, 2022
aa7dc32
update cli
Jan 10, 2022
0d962c8
use saturating sub
Jan 10, 2022
610cd46
tests
nope-finance Jan 11, 2022
da28ec9
add claim protocol fees
Jan 13, 2022
d657f7d
account checks
Jan 13, 2022
49fe464
ClaimProtocolFees -> ClaimReserveProtocolFees
Jan 13, 2022
06c9f8b
debug comments
Jan 13, 2022
5ccba68
formatting(again)
Jan 13, 2022
cac0a8a
comments
Jan 14, 2022
ba6b3c6
validate protocol liquidation fee
Jan 17, 2022
706777f
fix oopsie that i did earlier with update reserve config
Jan 17, 2022
caf51c0
tests
Jan 17, 2022
24b2b1b
formatting
Jan 17, 2022
4937344
make sure liquidations accumulate protocol fees
Jan 17, 2022
56e61d2
make liquidation protocol fee in terms of base liquidity token
Jan 19, 2022
2fd17da
adding claim check at end of liquidation test, cleaning up claim inst…
nope-finance Feb 17, 2022
17ac101
pr comments
nope-finance Feb 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
use saturating sub
  • Loading branch information
@nope-finance
ra authored and nope-finance committed Feb 17, 2022
commit 0d962c816c9a111df2a119682da6c43e37c0ecef
3 changes: 1 addition & 2 deletions token-lending/program/src/state/reserve.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -302,8 +302,7 @@ impl Reserve {
}

let withdraw_amount = withdraw_amount
.checked_sub(protocol_fee_amount)
.unwrap_or(0);
.saturating_sub(protocol_fee_amount);

Ok(CalculateLiquidationResult {
settle_amount,
Expand Down