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

symbiosis adds new chains (Ton, Bsquared, Morph, Cronos zkEvm) and uses generated config file. #12656

Closed
wants to merge 2 commits into from

Conversation

allush
Copy link
Contributor

@allush allush commented Dec 10, 2024

Dear Defillama team,

Symbiosis changed a way to deliver up-to-date config to defillama.

This config is updated based on actual version of Symbiosis SDK config https://github.com/symbiosis-finance/js-sdk/blob/main/src/crosschain/config/mainnet.ts

Thanks!

@llamatester
Copy link

The adapter at projects/symbiosis-finance exports TVL:

ethereum                  5.39 M
sei                       1.43 M
bsc                       825.63 k
polygon                   568.16 k
era                       545.05 k
arbitrum                  540.44 k
rsk                       434.78 k
merlin                    287.14 k
linea                     245.18 k
scroll                    240.97 k
boba_bnb                  219.75 k
manta                     208.31 k
bsquared                  201.66 k
ftn                       197.78 k
base                      186.14 k
gravity                   186.13 k
tron                      145.89 k
mantle                    143.76 k
optimism                  126.11 k
polygon_zkevm             116.62 k
boba                      114.17 k
cronos_zkevm              107.62 k
ton                       105.92 k
avax                      102.52 k
metis                     95.30 k
cronos                    88.62 k
mode                      57.98 k
taiko                     45.23 k
blast                     42.13 k
fraxtal                   40.77 k
arbitrum_nova             38.81 k
core                      36.77 k
zklink                    30.07 k
telos                     28.27 k
kava                      22.75 k
zeta                      19.53 k
morph                     0

total                    13.22 M 

@g1nt0ki
Copy link
Member

g1nt0ki commented Dec 10, 2024

 @allush thanks for the PR, refactored the code here, what is the source chain for CATI, SIS, LADYS, G?
 
 I am guessing that these are locked in one chain and minted in others

@g1nt0ki g1nt0ki self-assigned this Dec 10, 2024
@allush
Copy link
Contributor Author

allush commented Dec 11, 2024

@allush thanks for the PR, refactored the code here, what is the source chain for CATI, SIS, LADYS, G?    I am guessing that these are locked in one chain and minted in others

unfortunately, your refactoring will not allow us to update config easily. Our main point was to have generated config to be able to update it easily with new one generated by this tool https://github.com/symbiosis-finance/defillama-config-generator when something changed in our original SDK config. for your convenience we added comments for every token in this generated list. Could you please keep this config unmodified?

 I am guessing that these are locked in one chain and minted in others

According to our protocol it doesn't matter the way of emission of these tokens. Our protocol doesn't mint these tokens, just accept and lock them into Portal contract on corresponding chain as TVL and mint synthetic representation on BOBA BNB chain. Example of synthetic tokens minted by Symbiosis is here https://github.com/symbiosis-finance/js-sdk/blob/main/src/crosschain/config/cache/mainnet.json#L1522-L1569. But these synthetic tokens aren't included into generated config, that's why all of these is honest TVL of Symbiosis protocol.

@g1nt0ki
Copy link
Member

g1nt0ki commented Dec 11, 2024

got it, made a minor fix here, we dont remove deprecated chains, also, added config to ignore your file while we swap tokens with labels: #12665

@g1nt0ki g1nt0ki closed this Dec 11, 2024
@allush
Copy link
Contributor Author

allush commented Dec 11, 2024

got it, made a minor fix here, we dont remove deprecated chains, also, added config to ignore your file while we swap tokens with labels: #12665

thank you!

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.

4 participants