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 から始め、issue にリンクされている ApolloServer.ts のリファレンスとコンテキストの扱いを比較してください。コンテキストをチェックするメソッドと、オブジェクトを前提とするメソッドを確認し、指定された前提に沿ってそれらの動作を一貫させたうえで、影響を受けるチュートリアルのフローを検証してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- graphql, typescript
- 領域
- api, backend
- issue の種類
- リファクタリング
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100