-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat: add weETHs invariant event indexing #169
Conversation
66fb45a
to
248dedf
Compare
22d7e69
to
e558225
Compare
toAccount.effective_balance.plus( | ||
amountUnderlying.toBigDecimal().div(exponentToBigDecimal(18)), | ||
), | ||
); | ||
} | ||
} | ||
|
||
// eslint-disable-next-line @typescript-eslint/no-unused-vars |
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.
nit: not needed anymore
effective_balance: BigDecimal! | ||
"Token Address" | ||
token: Borrow! |
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.
I see EtherFi expects the balance to be in tokens instead of mantissa, what about renaming type
to token
? Would the new schema affect queries?
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.
line 52 is not referencing a value, It is referencing a relationship to make it easy to query all borrowers or suppliers.
line 51 is the name etherfi is expecting
e558225
to
29acca5
Compare
Add weETHs invariant support to etherfi subgraph