apollographql / apollographql/fullstack-tutorial

Assume context is an object

Aperta
#81 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
TypeScript
Stelle
1.2k
Fork
808
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia da final/server/src/datasources/user.js e confronta la gestione del contesto con il riferimento ApolloServer.ts collegato nell’issue. Esamina i metodi che verificano il contesto rispetto a quelli che assumono un oggetto, quindi rendi il loro comportamento coerente con l’ipotesi dichiarata e verifica il flusso del tutorial interessato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
graphql, typescript
Ambito
api, backend
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.