-
Notifications
You must be signed in to change notification settings - Fork 1
Graphql/issue #24 #35
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
base: development
Are you sure you want to change the base?
Conversation
Updated so the config for the query is locatead in source-plugin/gatsby-node.js. also ran a test query on the index.js Page
|
Hola @aag2807 tengo algunas dudas,
Por otro lado, hay algunos conflictos ya que se han fusionado algunos PRs. |
|
@mecm1993 Buenos dias , lo del plugin originalmente o tenia ubicado en el gatsby-node.js del root, despues de repasar https://www.gatsbyjs.com/tutorial/source-plugin-tutorial/#configure-an-apollo-client-to-fetch-data decidi tomar el approach que explica con el source plugin. Si les parece pudiera alocar= el codigo en el gatsby-node del root ota vez. soy tn tanto nuevo con los tests pero configurare la prueba antes de contaminar el UI, y solventare los merge conflicts, |
|
@aag2807 gracias por el link y la información, ya comprendo porque el acercamiento sobre el |
|
@mecm1993 disculpa el mention pero queria asegurarme si hacer el PR con el codigo en el gatsby-node.js del root, lo tengo listo y no deberia haber ningun merge conflict |
|
Hey @aag2807 no hay problema, en este caso, te diria que subas los cambios, si. Pero, ten en un branch aparte los cambios que aqui actualmente, de esa forma, si en futuro se quiere utilizar ya este disponible. |
Distinguidos @mecm1993 y @sperezm97 todavia no he tenido tiempo de estudiar la propuesta que nos planteo @aag2807 , pero creo que siempre tendremos algunas ventajas y desventajas con los distintas decisiones que tomemos con respecto a la arquitectura del proyecto. Por el momento no creo que la ubicacion de dicho paquete nos pueda ocacionar un problema en el futuro ya que podriamos trasladar la instalacion de dicho paquete a otra ubicacion mas adelante. |
removed source-plugin folder and updated gatsby-node.js in root
GraphQL Configuration
Created Source Plugin
Made sure queries would properly work in Graphql Playground
Made sure you could query from any page and use Data prop
made a mock layout to ensure the queries work