Rebalancing exchange issuance post audit improvements #500
Rebalancing exchange issuance post audit improvements #500
Conversation
Pull Request Test Coverage Report for Build 6195
💛 - Coveralls |
* The TokenFlush contains utility functions to send tokens and base SetTokens from the | ||
* Vault or Contract to a specified user address | ||
*/ | ||
contract TokenFlush is |
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.
Is this meant to be reused elsewhere? It's named like it's general but it seems highly coupled with the system. Would prefer it to be specific here instead if it's only done so we can test the library methods. RebalancingSetExchangeTokenChangeHandler
or something
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.
We previously had it in RebalancingSetIssuance
- but I realize that we will be using flushing functions for other modules in the future (e.g. token transformation pipeline). So that's why it's refactored to be generic.
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.
sg!
No description provided.