Skip to content

fix: apy numbers with updated address #1877

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

Merged
merged 3 commits into from
May 14, 2025
Merged

Conversation

neelra8n
Copy link
Contributor

@neelra8n neelra8n commented May 13, 2025

Yieldfi have migrated to new yUSD token, with this PR, we are updating the respective address to display correct yield numbers on platform

@llamatester
Copy link

The yieldfi adapter exports pools:

Test Suites: 1 passed, 1 total
Tests: 9 passed, 9 total
Snapshots: 0 total
Time: 0.249 s
Ran all test suites.

Nb of pools: 1
 

Sample pools:
┌─────────┬──────────────────────────────────────────────┬────────────┬───────────┬────────┬─────────────┬───────┬─────────┐
│ (index) │ pool                                         │ chain      │ project   │ symbol │ tvlUsd      │ apy   │ apyBase │
├─────────┼──────────────────────────────────────────────┼────────────┼───────────┼────────┼─────────────┼───────┼─────────┤
│ 0       │ '0x19Ebd191f7A24ECE672ba13A302212b5eF7F35cb' │ 'ethereum' │ 'yieldfi' │ 'YUSD' │ 12303627.17 │ 12.22 │ 12.22   │
└─────────┴──────────────────────────────────────────────┴────────────┴───────────┴────────┴─────────────┴───────┴─────────┘

@slasher125
Copy link
Collaborator

@neelra8n is this real yield? if so, pls remove apy and just keep apyBase. if not, remove apyBase and keep apy

@llamatester
Copy link

The yieldfi adapter exports pools:

Test Suites: 1 passed, 1 total
Tests: 9 passed, 9 total
Snapshots: 0 total
Time: 0.281 s
Ran all test suites.

Nb of pools: 1
 

Sample pools:
┌─────────┬──────────────────────────────────────────────┬────────────┬───────────┬────────┬─────────────┬───────┬─────────┐
│ (index) │ pool                                         │ chain      │ project   │ symbol │ tvlUsd      │ apy   │ apyBase │
├─────────┼──────────────────────────────────────────────┼────────────┼───────────┼────────┼─────────────┼───────┼─────────┤
│ 0       │ '0x19Ebd191f7A24ECE672ba13A302212b5eF7F35cb' │ 'ethereum' │ 'yieldfi' │ 'YUSD' │ 12511323.64 │ 12.17 │ 12.17   │
└─────────┴──────────────────────────────────────────────┴────────────┴───────────┴────────┴─────────────┴───────┴─────────┘

@neelra8n
Copy link
Contributor Author

neelra8n commented May 14, 2025

@slasher125 this is real yield, updated apyBase.

@llamatester
Copy link

The yieldfi adapter exports pools:

Test Suites: 1 passed, 1 total
Tests: 9 passed, 9 total
Snapshots: 0 total
Time: 0.259 s
Ran all test suites.

Nb of pools: 1
 

Sample pools:
┌─────────┬──────────────────────────────────────────────┬────────────┬───────────┬────────┬─────────────┬─────────┐
│ (index) │ pool                                         │ chain      │ project   │ symbol │ tvlUsd      │ apyBase │
├─────────┼──────────────────────────────────────────────┼────────────┼───────────┼────────┼─────────────┼─────────┤
│ 0       │ '0x19Ebd191f7A24ECE672ba13A302212b5eF7F35cb' │ 'ethereum' │ 'yieldfi' │ 'YUSD' │ 12511323.64 │ 12.17   │
└─────────┴──────────────────────────────────────────────┴────────────┴───────────┴────────┴─────────────┴─────────┘

@neelra8n neelra8n changed the title apy numbers with updated address fix: apy numbers with updated address May 14, 2025
@slasher125 slasher125 merged commit f827d96 into DefiLlama:master May 14, 2025
1 check passed
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