Skip to content

Commit

Permalink
correct safe approve all tokens and adjust the documentation (#1776)
Browse files Browse the repository at this point in the history
  • Loading branch information
sparrowDom committed Aug 30, 2023
1 parent 4d36852 commit d206ce4
Showing 1 changed file with 10 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -461,11 +461,15 @@ contract BalancerMetaPoolStrategy is BaseAuraStrategy {
}

/**
* @notice Approves the Balancer pool to transfer all supported
* assets from this strategy.
* Also approve any suppered assets that are wrapped in the Balancer pool
* like stETH and frxETH, to be transferred from this strategy to their
* respective wrapper contracts. eg wstETH and sfrxETH.
* @notice Approves the Balancer Vault to transfer poolAsset counterparts
* of all of the supported assets from this strategy. E.g. stETH is a supported
* strategy and Balancer Vault gets unlimited approval to transfer wstETH.
*
* If Balancer pool uses a wrapped version of a supported asset then also approve
* unlimited usage of an asset to the contract responsible for wrapping.
*
* Approve unlimited spending by Balancer Vault and Aura reward pool of the
* pool BPT tokens.
*
* Is only executable by the Governor.
*/
Expand All @@ -477,7 +481,7 @@ contract BalancerMetaPoolStrategy is BaseAuraStrategy {
{
uint256 assetCount = assetsMapped.length;
for (uint256 i = 0; i < assetCount; ++i) {
_approveAsset(assetsMapped[i]);
_abstractSetPToken(assetsMapped[i], platformAddress);
}
_approveBase();
}
Expand Down

0 comments on commit d206ce4

Please sign in to comment.