-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Added user level balance tracking for Yield Yak YRT tokens #5906
Added user level balance tracking for Yield Yak YRT tokens #5906
Conversation
Workflow run id 9017524852 approved. |
Workflow run id 9017525003 approved. |
Workflow run id 9017843771 approved. |
Workflow run id 9017843999 approved. |
Working on more significant changes to address "TOO MANY STAGES" issues |
… reduce number of stages
Workflow run id 9020830588 approved. |
Workflow run id 9020830845 approved. |
Okay looks like that last commit got it working |
Workflow run id 9020967033 approved. |
Workflow run id 9020967368 approved. |
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.
this works for me. thanks for continuing to write efficient & clean code!
This PR adds a temporal model for Yak Receipt Token (YRT) balances for the Yield Yak strategies, using
Transfer
events. It allows us to track wallet-level balances in each of the strategies over time, and in combination with the strategy-level balances model, will enable individual users to monitor their deposit token balances over time. It will also make the tracking of thetotalSupply
of any of these YRT tokens pretty straightforward too as we can just track the negative balance of the zero address.I've taken a similar approach to previous models (e.g.
yield_yak_balances.sql
macro), but if there are any questions on any of it please let me know.