feat: add home page #1
Conversation
PierreJeanjacquot
left a comment
PierreJeanjacquot
left a comment
There was a problem hiding this comment.
Nothing blocking, see comments for improvements
| import { graphql } from '@/graphql/gql'; | ||
|
|
||
| export const dealsQuery = graphql(` | ||
| query Deals($length: Int = 20, $skip: Int = 0) { |
There was a problem hiding this comment.
We need to include the @live directive in queries to thegraph network with refetchInterval
https://thegraph.com/docs/en/subgraphs/querying/graph-client/live
There was a problem hiding this comment.
I don't understand what this adds compared to the way I currently use it.
| import { graphql } from '@/graphql/gql'; | ||
|
|
||
| export const appsQuery = graphql(` | ||
| query Apps($length: Int = 20, $skip: Int = 0) { |
There was a problem hiding this comment.
We need to include the @live directive in queries to thegraph network with refetchInterval
https://thegraph.com/docs/en/subgraphs/querying/graph-client/live
| import { graphql } from '@/graphql/gql'; | ||
|
|
||
| export const datasetsQuery = graphql(` | ||
| query Datasets($length: Int = 20, $skip: Int = 0) { |
There was a problem hiding this comment.
We need to include the @live directive in queries to thegraph network with refetchInterval
https://thegraph.com/docs/en/subgraphs/querying/graph-client/live
There was a problem hiding this comment.
I don't understand what this adds compared to the way I currently use it.
| import { graphql } from '@/graphql/gql'; | ||
|
|
||
| export const taskQuery = graphql(` | ||
| query Tasks($length: Int = 20, $skip: Int = 0) { |
There was a problem hiding this comment.
We need to include the @live directive in queries to thegraph network with refetchInterval
https://thegraph.com/docs/en/subgraphs/querying/graph-client/live
There was a problem hiding this comment.
I don't understand what this adds compared to the way I currently use it.
| import { graphql } from '@/graphql/gql'; | ||
|
|
||
| export const workerpoolsQuery = graphql(` | ||
| query Workerpools($length: Int = 20, $skip: Int = 0) { |
There was a problem hiding this comment.
We need to include the @live directive in queries to thegraph network with refetchInterval
https://thegraph.com/docs/en/subgraphs/querying/graph-client/live
There was a problem hiding this comment.
I don't understand what this adds compared to the way I currently use it.
| query: TypedDocumentString<TResult, TVariables>, | ||
| ...[variables]: TVariables extends Record<string, never> ? [] : [TVariables] | ||
| ) { | ||
| const subgraphUrl = import.meta.env.VITE_POCO_SUBGRAPH_URL; |
There was a problem hiding this comment.
keep in mind that the subgraphUrl will depend on the displayed chain
There was a problem hiding this comment.
Indeed, the Multi-chain will be handled in another PR.
There was a problem hiding this comment.
copy button is widely used in preview tables, this may be suboptimal if we want the rows to be links to the detail page
There was a problem hiding this comment.
That's a very good point! I think the line is wide enough and we can keep it as is, but indeed, it would be interesting to get more feedback on this topic.
No description provided.