apollographql / apollographql/fullstack-tutorial
Assume context is an object
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.2k
- Forks
- 808
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne mit final/server/src/datasources/user.js und vergleiche die Verarbeitung des Kontexts mit der im Issue verlinkten Referenz ApolloServer.ts. Überprüfe die Methoden, die den Kontext prüfen, im Vergleich zu denen, die ein Objekt voraussetzen, stelle dann ihr Verhalten entsprechend der genannten Annahme konsistent her und verifiziere den betroffenen Tutorial-Ablauf.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- graphql, typescript
- Bereich
- api, backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100