-
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
track tornado fees #2695
track tornado fees #2695
Conversation
The tornado adapter exports:
|
@ReynardoEW could you covert to use llama sdk instead dune |
yes i will convert it to use sdk instead |
fees/tornado/index.ts
Outdated
} | ||
|
||
const getFees = async (options: FetchOptions) => { | ||
if (cachedData) { |
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.
can you change teh cachedData to be a map instead of single value, and cache key to be startTimestamp, else if we run the refill script, all the days would have the same data
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.
can you change teh cachedData to be a map instead of single value, and cache key to be startTimestamp, else if we run the refill script, all the days would have the same data
i'm planning to rewrite it to not use dune, will update it soon but for my next dune based adapter i will cache it that way
The tornado adapter exports:
|
i have changed it to use getlogs instead of dune |
looks good |
No description provided.