apollographql / apollographql/fullstack-tutorial
Assume context is an object
未关闭
- 主要语言
- 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