Bug in cancelTrip method from User datasource
@stemmlerjs ya está trabajando en esto.
Desde el 18/2/2020.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Due to this line cancelTrip throws an error.
{
"errors": [
{
"message": "Cannot read property 'flight_number' of undefined",
"locations": [
{
"line": 2,
"column": 3
}
],
"path": [
"cancelTrip"
],
"extensions": {
"code": "INTERNAL_SERVER_ERROR",
"exception": {
"stacktrace": [
"TypeError: Cannot read property 'flight_number' of undefined",
// ....
Following change will fix the error.
async cancelTrip({ launchId }) {
const userId = this.context.user.id;
const numberOfDeletedTrips = await this.store.trips.destroy({ where: { userId, launchId } });
return numberOfDeletedTrips !== 0;
}
As per the doc of destroy it returns number of deleted resources wrapped in Promise. So it should be resolved to get the count to decide if resources are deleted or not.
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.2k
- Forks
- 808
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apollographql/fullstack-tutorial
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
investigate
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
-
tutorial links are broken Abiertodocumentation
apollographql/fullstack-tutorial#285 · 3 comentarios · 1 asignado ·
-
investigate
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
apollographql/fullstack-tutorial#255 · 2 reacciones ·
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
apollographql/fullstack-tutorial#254 · 2 comentarios ·
Todos los issues de apollographql/fullstack-tutorial
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
modelcontextprotocol/inspector#2458 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
carbon-design-system/ibm-products#9907 ·