superfluid-finance / superfluid-explorer

The new and improved Superfluid Explorer, built with SDK-Core, SDK-Redux & Next.js & MaterialUI
https://explorer.superfluid.finance
MIT License
22 stars 28 forks source link

GDA fixes & project clean-up #198

Closed kasparkallas closed 9 months ago

vercel[bot] commented 10 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
superfluid-console ✅ Ready (Inspect) Visit Preview Jan 26, 2024 0:53am
currents-bot[bot] commented 10 months ago

Cypress Tests tests failed after 17m 3s

🟢 64 · 🔴 14 · ⚪️ 0 · 🟣 1

View Run Details

Run Details

- Project: Console - Groups: 1 - Framework: Cypress - Run Status: Failed - Commit: 318d6f132af188c6d24f2fdceaa1e9339e1f081f - Spec files: 11 - Overall tests: 78 - Duration: 17m 3s - Parallelization: 2

Failed Spec Files

| Spec File | Failures | | ------------- | ------------- | | [cypress/e2e/AccountPage.feature](https://app.currents.dev/instance/0Vqz91TTVqUZ) | 7 | | [cypress/e2e/PoolPage.feature](https://app.currents.dev/instance/qvdJR3VJeGB7) | 3 | | [cypress/e2e/IndexPage.feature](https://app.currents.dev/instance/WhLTsFLdbBhF) | 1 | | [cypress/e2e/IndexSubscriptionPage.feature](https://app.currents.dev/instance/af8lZSR9WOin) | 1 | | [cypress/e2e/PoolMemberPage.feature](https://app.currents.dev/instance/Arxf2DDsV8kI) | 1 | | [cypress/e2e/TokenPage.feature](https://app.currents.dev/instance/Dw3JCK04PKKx) | 1 |


This message was posted automatically by currents.dev | Integration Settings
elvijsTDL commented 9 months ago

Subgraphs not being up to latest schema on all networks currently breaking most of the GDA related pages , like here

[!NOTE]
Some tests need to be re-checked after subgraph schema is up to date on all networks , currently they will fail

image

vmichalik commented 9 months ago

pools should probably be identified by address right? CleanShot 2024-01-17 at 10 02 12@2x

kasparkallas commented 9 months ago

pools should probably be identified by address right?

You can see the address in the detail view. But sure, there's value in seeing it in the table to distinguish between the pools.

elvijsTDL commented 9 months ago

It does not seem to pick up the latest deployment GDA related stuff e.g. https://superfluid-console-git-gda-clean-up-superfluid-finance.vercel.app/bnb-smart-chain/accounts/0xF9Ce34dFCD3cc92804772F3022AF27bCd5E43Ff2?tab=pools , this wallet is an admin of multiple pools on BSC but none are shown , also the getNetFlow calls keep failing that need to be fixed Screenshot 2024-01-24 at 11 43 41