-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Echo Fees #2865
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
Echo Fees #2865
Conversation
The jellyverse adapter exports:
|
The echo adapter exports:
|
The echo adapter exports:
|
fees/echo/index.ts
Outdated
|
||
// Process each log, keeping only the platform fee portion | ||
for (const log of logs) { | ||
if (log.topics[2] !== ECHO_FEE_RECEIVER_TOPIC) continue; |
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 not pass this as part of topics? like here: https://github.com/DefiLlama/dimension-adapters/blob/master/fees/ether-fi/index.ts#L106
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.
fixed it
for (const log of logs) { | ||
if (log.topics[2] !== ECHO_FEE_RECEIVER_TOPIC) continue; | ||
const token = '0x' + log.topics[1].slice(26); | ||
const amount = BigInt(log.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.
why convert to BigInt and later convert to string? can leave it as is?
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.
ah, I see the check in line 49, I think dailyFees.add
accepts bigInt?
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.
fixed it
The echo adapter exports:
|
The echo adapter exports:
|
* add gmx v1 fees helper functions * add dailyVolume * track dailyRevenue for gmx v1 forks --------- Co-authored-by: g1nt0ki <[email protected]>
* add mellow LRTs fees adapter * add supply side revenue * update to reduce rpc calls * add backfill start time * minor fix
* fix fjord-foundry dexs v2 api * set kyberswap dead date for classic and elastic products
* query premia data using factories and blocks * remove console log
* refresh to new data api * refresh api
* vanilla finance perps * use log instead * use log instead * code refactor --------- Co-authored-by: osel <[email protected]> Co-authored-by: g1nt0ki <[email protected]>
Co-authored-by: CuongNT <[email protected]>
* add m0.org fees adapter * update start date * update correct start date
* feat: bucket fee & revenue statistic * code refactor --------- Co-authored-by: Jarek Lin <[email protected]>
* add typus fees * fix calculation * minor fix --------- Co-authored-by: g1nt0ki <[email protected]>
DexToro's perpetual DEX on Optimism has been discontinued, and the company has transitioned to an iOS trading app based on Solana. This commit replaces the code that was tracking the old Optimism app with updated code to track revenue (fees collected) on the project's Solana Treasury wallet.
The perpie adapter exports:
|
The 1delta adapter exports:
|
The dodo-agg adapter exports:
|
The enso adapter exports:
|
No description provided.