You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
nomenclature might be a bit confusion, but doing a bit of research on the react-f/e directory structure components living in `queries/` are those reading from on-chain, triggering txs, use to call subgraphs, endpoints etc (seems the "ideal" state for abstracting that logic) and then importing those in `components/`
however, what i realised regarding the "query" itself *.graphql files. it is that it could live in subgraphs/smartgarden-optimism-gauges/graphql (queries scope per subgraph) and then using the right config for codegen.yml. It could auto-generate the function helpers that we could import later on the queries/ files from subgraphs/smartgarden-optimism-gauges/generated/graphql.tsx. in that way all (gql, *.graphql) will be isolated per subgraph directory we have and then elsewhere could be imported from there
we will need to investigate if that could be create out of the box using, something from this or combination of:
to be frank, for me it feels more logical for the gql queries to live in the frontend folder, since they are part of the frontend. the subgraphs folders can then be thought of as the backend.
lets see, not sure i understand your codegen.yml idea.
however, what i realised regarding the "query" itself
*.graphql
files. it is that it could live insubgraphs/smartgarden-optimism-gauges/graphql
(queries scope per subgraph) and then using the right config for codegen.yml. It could auto-generate the function helpers that we could import later on thequeries/
files fromsubgraphs/smartgarden-optimism-gauges/generated/graphql.tsx
. in that way all (gql, *.graphql
) will be isolated per subgraph directory we have and then elsewhere could be imported from therewe will need to investigate if that could be create out of the box using, something from this or combination of:
Originally posted by @petrovska-petro in #25 (comment)
The text was updated successfully, but these errors were encountered: