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: AstroportClassic (outdated) -> deadFrom #11708

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

0xpeluche
Copy link
Contributor

TVL of AstroportClassic has been flat for several years, and there doesn't seem to be any activity on the protocol using the Terra blockchain anymore

@0xpeluche 0xpeluche self-assigned this Sep 23, 2024
@llamatester
Copy link

Error while running adapter at projects/astroport-classic:

Error: Failed to get https://terra-classic-lcd.publicnode.com/cosmwasm/wasm/v1/contract/terra1fnywlw4edny3vw44x04xd67uzkdqluymgreu7g/smart/eyJwYWlycyI6IHsgImxpbWl0IjogMzAgLCJzdGFydF9hZnRlciI6W3sidG9rZW4iOnsiY29udHJhY3RfYWRkciI6InRlcnJhMTR6NTZsMGZwMmxzZjg2enkzaHR5Mno0N2V6a2hudGh0cjl5cTc2In19LHsidG9rZW4iOnsiY29udHJhY3RfYWRkciI6InRlcnJhMW1kZGNkeDB1ang4OWYzOGd1N3pzcGsycjJmZmRsNWVueXoydTAzIn19XSB9fQ==
Truncated error stack:

    at get (/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/projects/helper/http.js:33:11)
    at async queryContract (/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/projects/helper/chain/cosmos.js:196:5)
    at async getAllPairs (/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/projects/terraswap/factoryTvl.js:26:21)
    at async /home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/projects/terraswap/factoryTvl.js:53:20
    at async getTvl (/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/test.js:52:23)
    at async /home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/test.js:171:9
    at async Promise.all (index 0)
    at async Promise.all (index 4)
    at async /home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/test.js:214:3

@0xpeluche 0xpeluche changed the title Fix: AstroportClassic (outdated) Fix: AstroportClassic (outdated) -> deadFrom Sep 23, 2024
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.

hmm, there might not be much activity, but still there is ton of tvl, need to look into why it is broken. And people are still trading there: https://app.astroport.fi/trade?poolAddress=terra1v3lqxl0eyte9x3nhdgcj8hwvjq76aupnnzz0yll8mxs5cckc29pqvg2scu

@0xpeluche
Copy link
Contributor Author

Isn't the trading you're showing actually for Terra2 instead? Because there is also an Astroport product https://defillama.com/protocol/astroport#information that is still functional

@g1nt0ki
Copy link
Member

g1nt0ki commented Sep 23, 2024

you are right, the pool I shared is terra2, we still need to figure out why the adapter for v1 is breaking now.

@g1nt0ki g1nt0ki merged commit 8840507 into DefiLlama:main Oct 4, 2024
@llamatester
Copy link

The adapter at projects/astroport-classic exports TVL:

terra                     0

total                    0 

Error saving cache: Error: ENOENT: no such file or directory, open '/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/node_modules/@defillama/sdk/build/util/local_cache/zlib-1.0/BlockCache.json'
    at Object.openSync (node:fs:596:3)
    at Object.writeFileSync (node:fs:2322:35)
    at process.saveCache (/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/node_modules/@defillama/sdk/build/util/cache.js:152:17)
    at process.emit (node:events:529:35)
    at process.exit (node:internal/process/per_thread:191:15)
    at /home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/test.js:251:11 {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/home/runner/work/DefiLlama-Adapters/DefiLlama-Adapters/node_modules/@defillama/sdk/build/util/local_cache/zlib-1.0/BlockCache.json'
}

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