apollographql / apollographql/fullstack-tutorial

Assume context is an object

Offen
#81 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.