Skip to content

initial list of cornucopia vaults #14177

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

Merged
merged 4 commits into from
Apr 7, 2025

Conversation

mshakeg
Copy link
Contributor

@mshakeg mshakeg commented Apr 3, 2025

Add Cornucopia Vaults to Canopy TVL


NOTE

Please enable "Allow edits by maintainers" while putting up the PR.


  • If you would like to add a volume adapter please submit the PR here.
  • If you would like to add a liquidations adapter, please refer to this readme document for details.
  1. Once your adapter has been merged, it takes time to show on the UI. If more than 24 hours have passed, please let us know in Discord.
  2. Sorry, We no longer accept fetch adapter for new projects, we prefer the tvl to computed from blockchain data, if you have trouble with creating a the adapter, please hop onto our discord, we are happy to assist you.
  3. Please fill the form below only if the PR is for listing a new protocol else it can be ignored/replaced with reason/details about the PR
  4. For updating listing info It is a different repo, you can find your listing in this file: https://github.com/DefiLlama/defillama-server/blob/master/defi/src/protocols/data2.ts, you can edit it there and put up a PR
  5. Do not edit/push package-lock.json file as part of your changes, we use lockfileVersion 2, and most use v1 and using that messes up our CI
  6. No need to go to our discord and announce that you've created a PR, we monitor all PRs and will review it asap

Name (to be shown on DefiLlama):
Twitter Link:
List of audit links if any:
Website Link:
Logo (High resolution, will be shown with rounded borders):
Current TVL:
Treasury Addresses (if the protocol has treasury)
Chain:
Coingecko ID (so your TVL can appear on Coingecko, leave empty if not listed): (https://api.coingecko.com/api/v3/coins/list)
Coinmarketcap ID (so your TVL can appear on Coinmarketcap, leave empty if not listed): (https://api.coinmarketcap.com/data-api/v3/map/all?listing_status=active,inactive,untracked&start=1&limit=10000)
Short Description (to be shown on DefiLlama):
Token address and ticker if any:
Category (full list at https://defillama.com/categories) *Please choose only one:
Oracle Provider(s): Specify the oracle(s) used (e.g., Chainlink, Band, API3, TWAP, etc.):
Implementation Details: Briefly describe how the oracle is integrated into your project:
Documentation/Proof: Provide links to documentation or any other resources that verify the oracle's usage:
forkedFrom (Does your project originate from another project):
methodology (what is being counted as tvl, how is tvl being calculated):
Github org/user (Optional, if your code is open source, we can track activity):

@llamabutler
Copy link

The adapter at projects/canopy exports TVL:

move                      323.62 k

total                    323.62 k 

@waynebruce0x
Copy link
Collaborator

Where can these vaults be accessed on your dapp UI?

@waynebruce0x waynebruce0x self-assigned this Apr 3, 2025
@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 3, 2025

Hey @waynebruce0x these vaults show up if the user has contributions in them

https://app.canopyhub.xyz/explore

Screenshot 2025-04-03 at 15 13 17

@waynebruce0x
Copy link
Collaborator

Why is TVL lower on your UI than in these stats?

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 3, 2025

@waynebruce0x those are for our test vaults. If you look at one of the actual vaults that I hardcoded in this PR you'll see it has ~320k in TVL

0x5652c4977b103d323756540dd030c2cd083539600f3ed07872fdb762f7c9ed4a

https://explorer.movementnetwork.xyz/object/0x5652c4977b103d323756540dd030c2cd083539600f3ed07872fdb762f7c9ed4a/coins?network=mainnet

@waynebruce0x
Copy link
Collaborator

Who is depositing into these vaults if theyre not on the UI?

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 3, 2025

@waynebruce0x

The depositors are participants in Movement's Cornucopia Campaign: movementnetwork.xyz/article/cornucopia-movement-liquidity-program-faq.

We are bridging liquidity from both movement.concrete.xyz and app.ether.fi/liquid/move-eth.

Once the liquidity is bridged, users will be able to claim their rewards for participating in the campaign through our UI and the bridged assets will then be deployed into Movement protocols via our Canopy vaults, where users can manage their positions.

@llamabutler
Copy link

The adapter at projects/canopy exports TVL:

move                      323.55 k

total                    323.55 k 

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 4, 2025

Hey @waynebruce0x we'd really like to this this PR merged in, is everything fine?

@waynebruce0x
Copy link
Collaborator

TVL should be counted on the chain where users make their deposits

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 4, 2025

Hey @waynebruce0x the TVL is bridged onto Movement. The users will then claim their assets on Movement.

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 4, 2025

@waynebruce0x you can see the liquidity that the cornucopia USDT-USDC vault deployed to the Meridian AMM pool on our UI

https://app.canopyhub.xyz/explore

Screenshot 2025-04-04 at 16 58 52

@waynebruce0x
Copy link
Collaborator

Yeah but they deposit into an ethereum (?) contract so in our methodology we require that the TVL should show there. See Rings protocol for example

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 4, 2025

@waynebruce0x but they bridged those assets over from ethereum to movement using LayerZero, so should it not show on the Movement TVL?

@waynebruce0x
Copy link
Collaborator

who bridged the assets? the user or the protocol?

@waynebruce0x
Copy link
Collaborator

assets bridged to movement will show as movement TVL when theyre deposited into defi protocols on movement

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 4, 2025

Hey @waynebruce0x, understood.

To clarify, our goal is for the bridged assets to count toward both Canopy’s TVL (as the aggregator and vault manager) and the underlying DeFi protocol we deploy into on Movement (e.g., Meridian AMM).

Users deposit into Ethereum-based vaults, but those vaults are explicitly designed to bridge liquidity over to Movement. From there, Canopy handles on-chain deployment into Movement-native protocols via our vaults. So while the deposit originates on Ethereum, the assets are programmatically bridged on behalf of users and are actively managed and deployed on Movement via LayerZero.

We believe it's appropriate for the TVL to reflect on both Canopy and the Movement DeFi protocols we integrate with.

Happy to make adjustments that better fit DefiLlama’s methodology while achieving this. Let us know what the best path forward would be here.

@llamabutler
Copy link

The adapter at projects/canopy exports TVL:

move                      323.48 k

total                    323.48 k 

@mshakeg
Copy link
Contributor Author

mshakeg commented Apr 5, 2025

Hey @waynebruce0x we've decided to remove Cornucopia from the TVL count for Canopy, so I've removed this from the adapter. Can you please review and let me know if it's good?

token1: parts[1],
curveType: parts[2]
};
} catch (error) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove any try/catch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@waynebruce0x ok, just removed it

@llamabutler
Copy link

The adapter at projects/canopy exports TVL:

move                      323.08 k

total                    323.08 k 

@waynebruce0x waynebruce0x merged commit 611c370 into DefiLlama:main Apr 7, 2025
1 check passed
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