Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Estimator needs to account for rewards that are claimed during operations #237

Open
PeterMPhillips opened this issue Aug 2, 2022 · 1 comment

Comments

@PeterMPhillips
Copy link
Member

Before we estimate the value of a strategy, we claim rewards. The estimator script needs to account for these claims to determine the value of the strategy before any deposits or withdraws are estimated.

@georgercarder
Copy link
Contributor

Goal is to refactor to only rely on the ControllerLibrary, that way the StrategyController doesn't have to be aware of the lens contract or do any checks

Refer to #99 to form a superior PR

To be done after the code4rena release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants