-
Notifications
You must be signed in to change notification settings - Fork 169
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
R02: Feature/pool 408 refactor credit accrual into prize pool #129
R02: Feature/pool 408 refactor credit accrual into prize pool #129
Conversation
/// @param amount The amount the user is withdrawing | ||
/// @param controlledToken The type of collateral the user is withdrawing (i.e. ticket or sponsorship) | ||
/// @return durationSeconds The duration of the timelock in seconds | ||
function _calculateTimelockDuration( |
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.
Inaccurate natspec docs for function signature
/// @notice Calculates the early exit fee for the given amount | ||
/// @param amount The amount of collateral to be withdrawn | ||
/// @return Exit fee | ||
function _calculateEarlyExitFee(address controlledToken, uint256 amount) internal 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.
Inaccurate natspec docs for function signature
/// @param _principal The principal amount on which interest is accruing | ||
/// @param _interest The amount of interest that must accrue | ||
/// @return durationSeconds The duration of time it will take to accrue the given amount of interest, in seconds. | ||
function estimateCreditAccrualTime( |
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 '_controlledToken' from natspec docs
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.
most functions appear to be missing this param from natspec docs
8e1f5db
to
1c36365
Compare
Move credit accrual from the prize strategy into the prize pool.