Skip to content

Add Ellipsis Finance adapter #3

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
May 7, 2022
Merged

Add Ellipsis Finance adapter #3

merged 4 commits into from
May 7, 2022

Conversation

@mashup-purveyor
Copy link
Contributor

@mashup-purveyor mashup-purveyor commented May 6, 2022

No description provided.

@slasher125
Copy link
Collaborator

slasher125 commented May 6, 2022

Hey sir, ty for the adaptor!

a few notes:

  • for project (use the defillama slug, which would be ellipsis-finance . Also use that for your project folder name
  • for chain: utils.formatChain('binance')
  • u don't have to round the apy value on the adaptor level, we handle that for you on the frontend so please remove that part. same for tvl, no need to round it

@mashup-purveyor
Copy link
Contributor Author

mashup-purveyor commented May 7, 2022

Hey thanks; all done.

@slasher125 slasher125 merged commit c2006b6 into DefiLlama:master May 7, 2022
@mashup-purveyor
Copy link
Contributor Author

mashup-purveyor commented May 7, 2022

Hey @slasher125,

Thanks for this. On the listings it isn't very apparent how to find these pools on the underlying dApp. For example, in the readme is states for symbol that we put, "symbol of the tokens in pool". So we did this, however without a pool name it is very hard for the user to go from an Ellipsis entry on the table and find that on our site.

I wonder if it wouldn't be useful to provide a URL to the pool, or at minimum a pool name?

tutacrypto added a commit to tutacrypto/defillama-yield-server that referenced this pull request May 25, 2022
@tutacrypto
fixed comments DefiLlama#1 and DefiLlama#3
344bf0f
@tutacrypto tutacrypto mentioned this pull request May 25, 2022
Merged
slasher125 pushed a commit that referenced this pull request May 27, 2022
@tutacrypto
angle (#17)
2e543ee 
* add-angle-adaptor

* change APY value to max boost

* updated api + changes in apy computation

* fixed comments #1 and #3

* changing the symbols so they fit defillama's framework, except for perps
slasher125 pushed a commit that referenced this pull request May 26, 2024
@Addis-Belete
feat: view all clearpool protocols pools in yields tab (#3) (#1315)
52160f0
slasher125 pushed a commit that referenced this pull request Mar 19, 2025
@bergben
Merge pull request #3 from DefiLlama/master
82a3ac9 
update
slasher125 pushed a commit that referenced this pull request Jul 3, 2025
@baonguyen1904
Merge pull request #3 from harmonixfi/feature/hyperevm-asset
5294dd9 
fix tvlUsd value
slasher125 pushed a commit that referenced this pull request Sep 24, 2025
@kraikov
Merge pull request #3 from yoprotocol/chore/move-yo-services-in-yo-ad…
1e67962 
…apter

chore: move yo services inside yo-protocol adapter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mashup-purveyor @slasher125