Skip to content
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

Adding Kamino Vault Revenue #1791

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kakagri
Copy link
Contributor

@kakagri kakagri commented Aug 20, 2024

This is a pull request to add fee data for Kamino Vaults.

It gives the only daily/total revenue (not fees) for Kamino vaults, data is based on this query from on flipside crypto: https://flipsidecrypto.xyz/kakagri/kamino-vault-fees-4cDBfs. However the results are served through cached data similarly to #1768. And served through this endpoint: https://allez-xyz--kamino-fees-api-get-fees-lifetime-kamino.modal.run/

@llamatester
Copy link

The kamino-liquidity adapter exports:

> [email protected] test
> ts-node --transpile-only cli/testAdapter.ts fees kamino-liquidity

🦙 Running KAMINO-LIQUIDITY adapter 🦙
---------------------------------------------------
Start Date:	Mon, 19 Aug 2024 00:59:35 GMT
End Date:	Tue, 20 Aug 2024 00:59:35 GMT
---------------------------------------------------

SOLANA 👇
Backfill start time: 11/10/2023
End timestamp: 1724115574 (2024-08-20T00:59:34.000Z)
Total revenue: 2.65 M
└─ Methodology: Revenues are aggregated by Allez Labs using the following flipside crypto: https://flipsidecrypto.xyz/kakagri/kamino-vault-fees-4cDBfs
Daily revenue: 3.77 k
└─ Methodology: Revenues are aggregated by Allez Labs using the following flipside crypto: https://flipsidecrypto.xyz/kakagri/kamino-vault-fees-4cDBfs

@dtmkeng
Copy link
Contributor

dtmkeng commented Aug 20, 2024

could you add dailyFees ?

@kakagri
Copy link
Contributor Author

kakagri commented Aug 21, 2024

we don't have the fees in the API yet, we plan to add it in the next weeks
can we add only the revenue in the meantime or do they need to be added at the same time ?

@dtmkeng
Copy link
Contributor

dtmkeng commented Aug 22, 2024

we don't have the fees in the API yet, we plan to add it in the next weeks can we add only the revenue in the meantime or do they need to be added at the same time ?

no can set rev = fees ?

@kakagri
Copy link
Contributor Author

kakagri commented Aug 22, 2024

we don't have the fees in the API yet, we plan to add it in the next weeks can we add only the revenue in the meantime or do they need to be added at the same time ?

no can set rev = fees ?

Fees are supposed to be what is earned by users no ?

@dtmkeng
Copy link
Contributor

dtmkeng commented Aug 22, 2024

Fees are supposed to be what is earned by users no ?

dailyFees: All fees and value collected from all sources, this also includes liquid staking rewards, generated yields and possible mint and burn fees paid by LP (but not transaction or gas fees).

@kakagri
Copy link
Contributor Author

kakagri commented Aug 22, 2024

so the fees are a bit different from the revenue, here I'm pushing only the revenue that the protocol is earning from the liquidity vaults

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants