-
Notifications
You must be signed in to change notification settings - Fork 1.5k
track treehouse fee + fix typo #4219
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
Conversation
The gauntlet.ts adapter exports:
|
The openeden-usdo.ts adapter exports:
|
The yearn-finance.ts adapter exports:
|
The treehouse-protocol adapter exports:
|
The gauntlet.ts adapter exports:
|
The openeden-usdo.ts adapter exports:
|
The yearn-finance.ts adapter exports:
|
The treehouse-protocol adapter exports:
|
The gauntlet.ts adapter exports:
|
The openeden-usdo.ts adapter exports:
|
The yearn-finance.ts adapter exports:
|
The treehouse-protocol adapter exports:
|
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.
.
fees/treehouse-protocol/index.ts
Outdated
const dailySupplySideRevenue = options.createBalances(); | ||
const dailyProtocolRevenue = options.createBalances(); | ||
const markedLogs = await options.getLogs({ | ||
target: ADDRESSES[options.chain as keyof typeof ADDRESSES].TREEHOUSE_ACCOUNTING, |
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.
one of the reasons why I hate typescript, instead of helping, it is making the code unreadable here, thats why I use any
everywhere. The solution here might be:
const { TREEHOUSE_ACCOUNTING, TREEHOUSE_TOKEN, ... } = (ADDRESSES as any)[options.chain]
Again, thanks for the PR @KPHEMRAJ
The gauntlet.ts adapter exports:
|
The openeden-usdo.ts adapter exports:
|
The yearn-finance.ts adapter exports:
|
The treehouse-protocol adapter exports:
|
Closes issue : #2928