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

fix zarban treasury #13310

Closed
wants to merge 11 commits into from
Closed

Conversation

Sajjad-Keshavarz
Copy link
Contributor

I’m sorry, there was a problem with merging. Some of the changes didn’t make it through the last merging pull request.

@llamabutler
Copy link

The adapter at projects/treasury/zarban.js exports TVL:

arbitrum                  1.02 k

total                    1.02 k 

Copy link
Member

@g1nt0ki g1nt0ki left a comment

Choose a reason for hiding this comment

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

rejecting this as this code does exactly the same thing

@g1nt0ki g1nt0ki closed this Jan 30, 2025
@Sajjad-Keshavarz
Copy link
Contributor Author

I noticed that the chain name before the token address is required in the ‘balances’ key. However, it seems to be missing in the current version of the code. This oversight causes the treasury TVL amount to be zero.

@g1nt0ki
Copy link
Member

g1nt0ki commented Feb 3, 2025

no, it is not needed if you use the api object

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