-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Events in HealthCheck #503
Comments
rareweasel
changed the title
Add Events in HealthCheck Changes
Add Events in HealthCheck
Dec 17, 2021
4 tasks
Hi @pandadefi are you planning to impl this? any ETA? I saw there is a merged pr but it looks like it is not what we needed for this issue. |
After 0.4.3 the heath check isn't on Base strategy but on vault to reduce Base strategy size. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add the corresponding events in the following functions:
Also, the
harvest()
function modifies the doHealthCheck here without emitting an event.Output
Emit events in both functions and in the harvest function where the
doHealthCheck
variables is modified.Target Tag/Branch
Since the health check is not present in the main branch, the target is v0.4.3
https://github.com/yearn/yearn-vaults/tree/v0.4.3
The text was updated successfully, but these errors were encountered: