DevsDO / whocode-frontend

frontend for the open source project inspired in http://indianswhodesign.in, made to expose all the developers from the Dominican Republic with their skills and contacts.
https://master.d3a2yvjk67i8de.amplifyapp.com/
BSD Zero Clause License
10 stars 1 forks source link

Graphql/issue #24 #35

Open aag2807 opened 4 years ago

aag2807 commented 4 years ago

GraphQL Configuration


Created Source Plugin

image

Made sure queries would properly work in Graphql Playground

image

Made sure you could query from any page and use Data prop

image

made a mock layout to ensure the queries work

mecm1993 commented 4 years ago

Hola @aag2807 tengo algunas dudas,

Por otro lado, hay algunos conflictos ya que se han fusionado algunos PRs.

aag2807 commented 4 years ago

@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,

mecm1993 commented 4 years ago

@aag2807 gracias por el link y la información, ya comprendo porque el acercamiento sobre el source-plugin. En este caso, entiendo que no es necesario tenerlo ya que el objetivo es consumir data en vez de servir como fuente de datos. Aunque quisiera tener la opinión de @sperezm97 y @dionicio-gp sobre el tema para así saber cual sería el acercamiento pertinente.

aag2807 commented 4 years ago

@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

mecm1993 commented 4 years ago

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.

dioniciodiaz commented 4 years ago

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.