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
Updated DrawManager events to include reward #8
Conversation
LCOV of commit
|
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.
just the usual natspec upkeep!
event DrawStarted( | ||
address indexed sender, | ||
address indexed recipient, | ||
uint24 drawId, | ||
uint32 rngRequestId, | ||
uint48 elapsedTime | ||
uint24 indexed drawId, | ||
uint48 elapsedTime, | ||
uint reward, | ||
uint32 rngRequestId | ||
); |
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 updates
src/DrawManager.sol
Outdated
/// @notice Emitted when the finish draw is called | ||
/// @param drawId The draw id | ||
/// @param elapsedTime The amount of time that had elapsed between start draw and finish draw | ||
/// @param startRecipient The recipient of the start rng auction reward | ||
/// @param startReward The reward for the start rng auction | ||
/// @param finishRecipient The recipient of the finish draw auction reward | ||
/// @param finishReward The reward for the finish draw auction | ||
/// @param recipient The recipient of the finish draw auction reward | ||
/// @param reward The reward for the finish draw auction | ||
/// @param remainingReserve The remaining reserve after the rewards have been allocated | ||
event DrawFinished( | ||
address indexed sender, | ||
address indexed recipient, | ||
uint24 indexed drawId, | ||
uint elapsedTime, | ||
address indexed startRecipient, | ||
uint startReward, | ||
address indexed finishRecipient, | ||
uint finishReward, | ||
uint48 elapsedTime, | ||
uint reward, |
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 some natspec updates
LCOV of commit
|
No description provided.