apollographql / apollographql/fullstack-tutorial

Assume context is an object

未关闭
#81 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
TypeScript
星标
1.2k
派生
808
PR 合并指标
30 天内没有已合并 PR

描述

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

贡献指南

这个仓库没有索引到贡献指南

调研方向

从 final/server/src/datasources/user.js 开始,将其 context 处理方式与 issue 中链接的 ApolloServer.ts 参考实现进行比较。检查用于验证 context 的方法,以及那些假定其为对象的方法,然后根据所述假设使它们的行为保持一致,并验证受影响的教程流程。

由索引模型根据 Issue 内容生成。

评估

技术栈
graphql, typescript
领域
api, backend
Issue 类型
重构
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。