Paolo Donato Torregrosa Navarro
Paolo Donato Torregrosa Navarro
Fernando Cordero
Juan Jose Vega
silvana murgo
Julian Franco
César David Ramírez Dimaté
Reinaldo Mendoza
Steve Anthony Luzquiños Agama
José Vinicio Flores Arias
David Behar
IT Desarrollo invertirOnline.com
David Behar
Angelo Zambrano
Jairo Campos Ruiz
Miguel Angel Reyes Moreno
Alex Paul Chirino Caicedo
Miguel Angel Reyes Moreno
Danilo Valenzuela
Reinaldo Mendoza
Andrés Felipe Castañeda
Jhon Manuel Angulo Moncada
Fernando Cordero
César David Ramírez Dimaté
Platzi Team
Daniel Esteves
Diego Fernando Caviedes Camaho
Pablo Estornut
Jesus Rodriguez
Cristian David Franco Garcia
César David Ramírez Dimaté
Jose Eduardo Romero Baltazar
Alejandro Cadena Florián
Reinaldo Mendoza
Yerlinson Maturana Serna
![Francisco Garcia [C6]](https://static.platzi.com/media/avatars/avatars/garciafran_c845c2fe-2ea4-42db-a305-a7f4ef13b0fd.jpeg)
Francisco Garcia [C6]
Este curso no lo quería tomar al principio pero resulta ser realmente interesante, tome la primera clase y ahora quiero hacerlo todo! no conocía a este profesor tan bueno
No se porque pero no a mi no me devuelve el arreglo de people con información, siempre lo muestra vació. Revise el código por todos lados, compare con el código de los enlaces. Tengo todo igual, alguno tiene idea de porque me esta pasando esto?
tiene que haber un error en addPeople Ya revisaste si la DB los guardo?
A mi me pasaba lo mismo y empece a debugger con console.log los ID que insertaba y trataba de buscar. Y mi error era que cuando guardaba la relacion en addPeople ponia 2 veces el id del curso :face_with_head_bandage:
Fix
addPeople: async (root, { courseID, personID }) => { let db, course, person; console.log(`courseid ${courseID}`); console.log(`pid ${personID}`); try { db = await connectDB(); course = await db .collection('courses') .findOne({ _id: ObjectID(courseID) }); console.log(course); person = await db .collection('students') .findOne({ _id: ObjectID(personID) }); console.log(person); if (!course || !person) { throw new Error("Person or Course doesn't exists"); } await db .collection('courses') .updateOne( { _id: ObjectID(courseID) }, { $addToSet: { people: ObjectID(personID) } } ); } catch (error) { console.error(error); } return course; },
Un refactor para el resolver de people para no abrir conexion a la base de datos si no es necesario y no hacer ternarias:
const peopleResolver = async ({ people }) => { try { if (!people || people.length === 0) return []; const db = await connectDB(); const ids = people.map(id => ObjectID(id)); const peopleData = await db .collection('students') .find({ _id: { $in: ids } }) .toArray(); return peopleData; } catch (err) { console.log(`Err in people types: ${err.message}`); } };
Aca verificamos si hay people en la collection antes de abrir la conexion y retornamos el Array vacio de una vez. Tambien nos ahorramos las ternarias dentro de la funcion. Coloque la excepcion de people.length === 0 pensando a futuro la posibilidad de remover una persona de people que dejaria un Array vacio en la base de datos.
Me suena. Sobretodo por el tema de no tener que abrir una conexión a base de datos innecesaria.
Muy buena la idea, solo abrirla si es estrictamente necesario
Quizás esta explicación le se útil a alguien, graphql tools va a intentar popular el campo students, ya que este campo es un nested type.
Este intento de popular el tipo Course se llama resolución de tipos. En "Cristiano", el resolver un tipo consiste en ejecutar una callback del mismo nombre que el nested type, para así "llenar" el campo people con "algo", este "algo" debe ser un array del tipo Student (también sería válido un array vacío).
Es debido a este comportamiento que cuando el profesor le pide a GraphQL el campo people, GraphQL puede devolverle un array con objetos del tipo Student, y si no hay ningún estudiante matriculado en el curso, un array vacío.
Y aqui esta el alcance de GraphQL!
Exacta mente qué son los resolvers, esta clase me confundió porque estamos agregando los tipos a la raíz de los resolvers, osea
const resolvers = { ... Course { ... } }
En realidad lo que estas agregando es el resolvers de los types que tienes definido en tu schema.
Eso qué significa, jeje 😅
Creo que todo este código se puede simplificar con el método de Mongoose populate(<Field>):
/* TypeScript */
Por lo que puedo ver creo que no es exactamente porque si tu haces populate siempre va a buscar la información de cada estudiante pero si lo haces de esta manera solo irá a buscarlo en caso de que se le pida mediante la consulta de graphql.
Con lo cual si en la consulta no se pide la info de los estudiantes te ahorras mucho trabajo de la base de datos buscando esta info. Esto en una aplicación con muchas peticiones se nota.
¿Por qué hicimos ...types? Me quedó esa duda.
Los 3 puntos (...) en JS se encarga de desempaquetar (deconstruir) un arreglo u objeto.
Por ejemplo, si tu objeto padre es:
const parent = { childOne: 'This is a string', }
Y luego tienes un objeto hijo (cuyos valores quieres agregárselos al padre) como el siguiente:
const children = { childTwo: 'Another string', childThree: 'Another one', }
Puedes hacerlo utilizando la sintaxis de los 3 puntos (...):
const parent = { childOne: 'This is a string', ...children, }
Que es equivalente a que el objeto padre lo declares manualmente:
const parent = { childOne: 'This is a string', childTwo: 'Another string', childThree: 'Another one', }
Excelente 😎
¿Cómo puedo hacer un resolver para types sin necesidad de GraphQL tools?
Por lo que dice el profesor sera mas complicado
GraphQL es lo mejor que he visto de los cursos que llevo hasta ahora!
excelente funcionalidad, aplicable para la mayoría de proyectos.
'use strict' const connectDb = require('./db') const { ObjectID } = require('mongodb') module.exports = { Course: { people: async ({ people }) => { let db let peopleData let ids try { db = await connectDb() ids = people ? people.map(id => ObjectID(id)) : [] peopleData = ids.lenght > 0 ? await db.collection('students').find( { _id: { $in: ids} } ).toArray() : [] } catch (error) { console.log(error) } return peopleData } } }```
Tienes mal escrito lenght, es length.
Como se puede resolver un tipo escalar, por ejemplo date, datatime, binary, entre otros
gracias
Hola! Debes crear un archivo de tipo Scalar.graphql y guardar los tipos de datos que necesites; más información aquí: https://github.com/danestves/GraphQL-CMS-Backend/tree/master/src/graphql/types
que plugin utiliza para mongo en vsCode?
Después de aplicar los cambios de este video, al hacer el mutation addPeople, no me devuelve el curso con people correcto sino un array vacío en el momento de ejecutar por primera vez el mutation, después sí.
asegurate de que esta linea sea correcta
ids = people ? people.map( id => ObjectID(id))
En mutations.js agregue las siguientes instrucciones para que este método nos retorne el curso con sus nuevos cambios(id de la persona) y así no obtener campos nulos de person la primera vez que se agrega una persona al curso.
Entiendo. Cuando ejecutras
mutation { addPeople( courseID: "60ee342200a30f84ee38faaf" personID: "60983f4961f3eb28d34942df" ) { _id title } }
Es para que no devuelva
{ "data": { "addPeople": null } }
sino
{ "data": { "addPeople": { "_id": "60ee342200a30f84ee38faaf", "title": "Curso de ejemplo 1" } } }
De igual manera puedes seguir usando la variable course, no hay necesidad de crear updateCourse sino vuelves a llamar la consulta en mongo y la asignas a course.
Woow esta clase ah sido la mejor, no sabia el potencial de graph
No sé si hice algo mal previamente, o ha cambiado node.js, pero en la parte de
people.map(id => ObjectID(id))
no me funciona a mi, así que asumí que estaba tomando a id como un objeto e hice
people.map(id => ObjectID(id._id))
Codigo completo de esa pieza (el codigo es un tanto distinto al del profesor porque hace parte de una funcion reutilizable para no estar llamando a la db cada rato):
case "mapArrayInCollection": // Tomaremos los ids del array al cual mapearemos. Ejemplo: En Course, queremos mapear el array de Students, esto sera un array de _id de students. let ids = relation != null ? relation.map(id => ObjectID(id._id)) : []; // Tomaremos de la base de datos los valores correspondientes y los devolveremos como el respectivo array. let arrayData = ids.length > 0 ? await collection.find({_id: {$in: ids}}).toArray() : []; return arrayData;
en realidad es para convertir id a un objeto id de mongo, esta dificil saber cual es el error porque no esta el contexto para saber que respresenta cada variable, map es una funcion superior de js, recorre todo un array aplicando la funcion que le pases como parametro a cada uno de los elementos
'use strict' const connectDB = require('./db'); const { ObjectID } = require('mongodb') const types = { Course: { people: async ({ people }) => { try { const db = await connectDB() const ids = people ? people.map(id => ObjectID(id)) : [] const peopleData = ids.length > 0 ? await db.collection('students').find( { _id: { $in: ids } } ).toArray() : [] return peopleData } catch (error) { console.error(error) } } } } module.exports = types```
Muy interesante :-)
types.js
'use strict' const connectDb = require('./db') const { ObjectID, ObjectId } = require('mongodb') module.exports = { Course: { people: async ({ people }) => { let db let peopleData let ids try { db = await connectDb() ids = people ? people.map(id => ObjectId(id)) : [] peopleData = ids.length > 0 ? await db.collection('students').find({ _id: { $in: ids } }).toArray() : [] } catch (error) { console.error(error) } return peopleData } } }
resolvers.js
'use strict' const mutations = require('./mutations') const queries = require('./queries') const types = require('./types') module.exports = { Query: queries, Mutation: mutations, ...types }