-
Notifications
You must be signed in to change notification settings - Fork 32
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
Added ability to donate to the prizes #92
Added ability to donate to the prizes #92
Conversation
LCOV of commit
|
@@ -230,6 +230,8 @@ contract PrizePool is TieredLiquidityDistributor, Ownable { | |||
/// @notice Records the last shutdown withdrawal for an account | |||
mapping(address vault => mapping(address user => uint24 drawId)) internal _lastShutdownWithdrawal; | |||
|
|||
address public constant DONATOR = 0x000000000000000000000000000000000000F2EE; |
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.
missing natspec
Also would be nice to know why this address was chosen.
@@ -355,6 +357,11 @@ contract PrizePool is TieredLiquidityDistributor, Ownable { | |||
return _deltaBalance; | |||
} | |||
|
|||
function donatePrizeTokens(uint256 _amount) external { |
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.
missing natspec
src/PrizePool.sol
Outdated
@@ -579,7 +586,7 @@ contract PrizePool is TieredLiquidityDistributor, Ownable { | |||
address _vault, | |||
uint24 _startDrawIdInclusive, | |||
uint24 _endDrawIdInclusive | |||
) external view returns (uint256) { | |||
) public view returns (uint256) { |
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.
doesn't need to be public
src/PrizePool.sol
Outdated
function getDonatedBetween( | ||
uint24 _startDrawIdInclusive, | ||
uint24 _endDrawIdInclusive | ||
) public view returns (uint256) { |
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.
doesn't need to be public
LCOV of commit
|
No description provided.