Conversation
|
The concrete-v1 adapter exports: |
|
The concrete-v1 adapter exports: |
|
The concrete-v2 adapter exports: |
noateden
left a comment
There was a problem hiding this comment.
can you try to get revenue from fee instead, do not use token received? try to find functions from vault contracts to get performance and management fees if any.
|
The concrete-v1 adapter exports: |
|
The concrete-v2 adapter exports: |
done |
|
hi @KPHEMRAJ, I tested and got all 0 fees. Can you check? |
|
The concrete-v1 adapter exports: |
|
The concrete-v2 adapter exports: |
fees/concrete-v1/index.ts
Outdated
|
|
||
| const vaultsResponse = await getConfig('concrete', `${CONCRETE_API_URL}/vault:tvl/all`); | ||
|
|
||
| const vaults = new Set(Object.values(vaultsResponse[currentChainId]).filter((vault: any) => vault.version === 1 && +vault.peak_tvl > 0).map((v1Vault: any) => v1Vault.address)); |
There was a problem hiding this comment.
can keep peak_tvl > 10k to filter out low value pools?
fees/concrete-v2/index.ts
Outdated
|
|
||
| const vaultsResponse = await getConfig('concrete', `${CONCRETE_API_URL}/vault:tvl/all`); | ||
|
|
||
| const vaults = new Set(Object.values(vaultsResponse[currentChainId]).filter((vault: any) => vault.version === 2 && +vault.peak_tvl > 0).map((v2Vault: any) => v2Vault.address)); |
g1nt0ki
left a comment
There was a problem hiding this comment.
since we have a single listing for concrete, can you merge it into single adapter here: https://defillama.com/protocol/concrete
|
The concrete-v1 adapter exports: |
|
The concrete adapter exports: |
|
The concrete adapter exports: |
* track concrete fees * smol fix * use functions instead of evnnt logs to calculate performance and management fees * fix start date * fix calculations * merge both versions * remove redundant code --------- Co-authored-by: Eden <noat.eth@gmail.com>
Closes : #4791