PE-7115: remove use of delegates field on gateways and update app to use getGatewayDelegates() and getDelegations() #111
Conversation
…e getGatewayDelegates() and getDelegations()
|
Visit the preview URL for this PR (updated for commit 0e3a9b2): https://ar-io-network-portal-a40ee--pr111-pe-7115-update-to-re-w2mceo55.web.app (expires Thu, 28 Nov 2024 22:45:45 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7abfae09c4446982a71cbb94b0cbf4688377a111 |
src/components/Header.tsx
Outdated
| value={ | ||
| gateways | ||
| ? Object.entries(gateways).filter( | ||
| (entry) => entry[1].status === 'joined', |
There was a problem hiding this comment.
| (entry) => entry[1].status === 'joined', | |
| ([_address, gateway]) => gateway.status === 'joined', |
There was a problem hiding this comment.
I tried something like this initially but I get a lint error on unused _address. I went ahead with the change but added a //eslint-disable
|
|
||
| return retVal; | ||
| }, | ||
| staleTime: Infinity, |
There was a problem hiding this comment.
any consequence of this stale time?
There was a problem hiding this comment.
I added this in so that the query should return the same value for the duration of the session unless explicitly invalidated by user action (i.e., instant withdrawal, cancelling a stake, adding new stake).
| @@ -76,8 +76,9 @@ const Header = () => { | |||
| value={ | |||
| gateways | |||
| ? Object.entries(gateways).filter( | |||
| (entry) => entry[1].status === 'joined', | |||
| ).length | |||
| // eslint-disable-next-line @typescript-eslint/no-unused-vars | |||
There was a problem hiding this comment.
i thought maybe the _ would avoid needing this, fine to revert
|
Related PR: ar-io/ar-io-network-process#182 |
No description provided.