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

Limitless TVL #10257

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Limitless TVL #10257

wants to merge 5 commits into from

Conversation

jihyunsong42
Copy link
Contributor

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

Hi, I've checked your refactored code, thanks.
but I think TVL is not shown properly as it supposed to. Can you check if there's anything missing?

tvl tvlll

@llamatester
Copy link

The adapter at projects/limitless exports TVL:

base                      145.32 k
arbitrum                  0
linea                     0

total                    145.33 k 

@g1nt0ki
Copy link
Member

g1nt0ki commented May 17, 2024

hmm, had a brief look, not sure where the issue is, will go through the code again next week

@g1nt0ki g1nt0ki self-assigned this May 17, 2024
@jihyunsong42
Copy link
Contributor Author

Hi, I am just wondering if you figured out the issue. If you have any questions please let me know

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.

None yet

3 participants