search

steve0xp / yearn_mellow-gearbox-strategy

yearn strategy depositing vault assets into mellow protocol gearbox strategy(ies)
GNU Affero General Public License v3.0
1 stars 0 forks source link

Add `forceHarvestTriggerOnce` #17

Closed steve0xp closed 1 year ago

steve0xp commented 1 year ago

From Val John:

Also, for the harvest triggers, maybe we should consider setting forceHarvestTriggerOnce to true, so it accounts for gas price.

Setting the trigger to true when we want to claim profit, or at least including the basefee in the logic to consider gas price.

steve0xp commented 1 year ago

Closed as this feature was added in PR #18