-
Notifications
You must be signed in to change notification settings - Fork 7
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
KUSD mint cap #346
KUSD mint cap #346
Conversation
f7b2236
to
b8c472a
Compare
@@ -684,6 +685,10 @@ contract BorrowerOperations is KumoBase, CheckContract, IBorrowerOperations { | |||
_kusdToken.burn(_account, _KUSD); | |||
} | |||
|
|||
function KUSDMintRemainder(address _asset) external view returns (uint256) { | |||
return kumoParams.KUSDMintCap(_asset) - kusdToken.totalSupply(); |
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.
@arthurka-o - do we need to check for a negative return value? My concerns are if fees could play in here. What do you think?
# Conflicts: # packages/contracts/contracts/BorrowerOperations.sol # packages/contracts/contracts/Interfaces/IKumoParameters.sol # packages/lib-ethers/abi/KumoParameters.json # packages/lib-ethers/test/MultiAssetHintsAdjustTrove.test.ts # packages/lib-ethers/types/index.ts
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.
LGTM, @arthurka-o.
Before I approve, can you please list all test cases in bullet points that you covered for the mint cap here?
e.g.: - KUSD > mintCap --> Does not mint new KUSD and sends error message EXY
BorrowerOperations > withdrawKUSD --> not minting new KUSDToken if the mint cap is reached |
Thanks, @arthurka-o
|
|
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.
LGTM!
No description provided.