beefyfinance / beefy-api

Simple API for BeefyFinance
https://api.beefy.finance
MIT License
118 stars 199 forks source link

Add USDC-STAR stable pool on Base Aerodrome #1492

Closed ying13 closed 4 months ago

ying13 commented 4 months ago

I want to add a USDC-STAR vault to app but the STAR token does not exist. What would be a better way to develop the vault without publishing updates to the address book?

roman-monk commented 4 months ago

star unsafu iirc

ying13 commented 4 months ago

@roman-monk Thanks for the review and help. Is the reason not enough liquidity or no timelock? I see the market cap and volume are small. I checked the token contracts but not certain if there’s migrate or upgrade functionality. 

Out of curiosity, what else we need to update to add a new token? I deployed the contracts, updated the pools and address book in beefy-api, vault data and images in beefy-v2, then ran both repositories, I saw the new token icon was shown in the localhost app, but the values like TVL, APY are 0. Was that because the address book was not published? 

kexleyBeefy commented 4 months ago

@ying13 STAR's largest holder (61%) is the team's multi-sig, that needs to be timelocked or at least be controlled in some way. Also unlike most tokens it is a debt token that can be minted through various contracts using price feeds, all of which need to be secured. It is quite hard to prove secureness on these kind of tokens without working with their team to check everything.

For some untimelocked tokens we can use the [NO_TIMELOCK] flag. But that is reserved for tokens which have large marketcap, huge demand and the team is well-known. But even then they have to be checked throughly and have very few areas of vulnerability.

Did you change the URL in this line to http://localhost:3000? That's how to get it look at your local API instead of the prod one. The APY and TVL should appear but without the address book package update you shouldn't see the info about the token at the bottom or able to zap in to the vault.

ying13 commented 4 months ago

@kexleyBeefy Thank you very much for the explanation! Yes I changed the beefy-api API_URL to http://localhost:3000 and the APY and TVL are 0 as below, and there's no zap feature. Screen Shot 2024-07-04 at 09 48 29

ying13 commented 4 months ago

I went over this process wanted to know how to verify and add new tokens. I will close this PR and many thanks to @kexleyBeefy @roman-monk for your time providing the helpful information 🙏