-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 velo superchain bribes, volume/fees #2321
Conversation
The velodrome-slipstream adapter exports:
|
The velodrome-v2 adapter exports:
|
The velodrome-v2 adapter exports:
|
}) | ||
return { |
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.
why return only dailyBribes
?
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 thought that the other metrics were already returned in dexs
. Should I add back dailyFees
dailyRevenue
and dailyHoldersRevenue
here?
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.
we track by adapter if you remove velodrome v2 is can't show fees
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.
Added it back
The velodrome-slipstream adapter exports:
|
The velodrome-v2 adapter exports:
|
The velodrome-v2 adapter exports:
|
Added bribes fetching from leaf Voter contract on the Superchain L2s for Velodrome. Added swaps fetching for the Superchain deployments on Mode + Lisk + Fraxtal. Left the Soneium and Ink deployments commented out for now because the backend doesn't seem to handle them yet