-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
@@ -36,6 +38,11 @@ import { Rebalance } from "../core/lib/Rebalance.sol"; | |||
* | |||
* A helper contract that mints a cToken from its underlying or redeems a cToken into | |||
* its underlying used for bidding in the RebalanceAuctionModule. | |||
* | |||
* CHANGELOG: | |||
* - Remove reentrant modifier on bidAndWithdraw. This modifier is already used in RebalanceAuctionModule |
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.
Would be good to add month/date/year of change
* Bid on rebalancing a given quantity of sets held by a rebalancing token wrapping or unwrapping | ||
* a target cToken involved. The tokens are returned to the user. This function is only compatible with | ||
* Rebalancing Set Tokens that use TWAP liquidators | ||
* |
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.
Would be good to add some language like here to note the exact problem this function is trying to solve.
“During a TWAP chunk auction, there is an adverse scenario where a bidder submits a chunk auction bid with a low gas price and iterateChunkAuction is called before that transaction is mined. When the bidder’s transaction gets mined, it may execute at an unintended price. To combat this, the BidAndWithdrawTWAP function checks that a new chunk auction has not been initiated from the point of bidding. The intended use case is that the bidder would retrieve the Rebalancing SetToken’s lastChunkAuctionEnd variable off-chain and submit it as part of the bid.”
Pull Request Test Coverage Report for Build 8274
💛 - Coveralls |
LGTM |
No description provided.