apollographql / apollographql/fullstack-tutorial
Assume context is an object
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.2k
- Forks
- 808
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Some methods check `this.context` and some assume it's an object:
https://github.com/apollographql/fullstack-tutorial/blob/master/final/server/src/datasources/user.js
I think it's safe to assume it's an object:
https://github.com/apollographql/apollo-server/blob/875944ea8358caa2140e6f7fff861f7a8f0149e4/packages/apollo-server-core/src/ApolloServer.ts#L587
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza con final/server/src/datasources/user.js y compara su manejo del contexto con la referencia ApolloServer.ts enlazada en el issue. Revisa los métodos que comprueban el contexto frente a los que asumen un objeto; después, haz que su comportamiento sea coherente con la suposición indicada y verifica el flujo del tutorial afectado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- graphql, typescript
- Área
- api, backend
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100