Updated YieldBricks adapter to show stats more correctly #16740
Conversation
|
The adapter at projects/yieldbricks exports TVL: |
projects/yieldbricks/index.js
Outdated
| tokens: TOKENS_YBR_ARBITRUM, | ||
| tvl: sumTokensExport({ | ||
| owners: ARB_STAKING_POOLS, | ||
| tokens: ARB_CONTRACTS_YBR, |
There was a problem hiding this comment.
any YBR TVL should be exported as staking like it was previously
There was a problem hiding this comment.
Ok, how do we calculate the TVL ?
There was a problem hiding this comment.
If you have no other TVL then use tvl: () => ({})
There was a problem hiding this comment.
I do have tvl, that is why I would like it to be added to the tvl value. If I use tvl: () => ({}) then tvl will show wrong value (0 value)
There was a problem hiding this comment.
any YBR TVL is exported as staking and will show as TVL depending on end user settings. Any core TVL can be added to this adapter with further changes
There was a problem hiding this comment.
@waynebruce0x I have now completed the changes that you commented and cleaned up the implementation. Please review again.
There was a problem hiding this comment.
Thanks, looks good, one last thing, please move the YBR addresses from coreAssets file to inside your adapter file
There was a problem hiding this comment.
@waynebruce0x - all done 🫡
|
The adapter at projects/yieldbricks exports TVL: |
|
The adapter at projects/yieldbricks exports TVL: |
|
@waynebruce0x please review |
3 participants
Updated YieldBricks adapter to show stats more correctly. Added LP stats and ethereum as second chain.