Closed kuniyuki-f closed 1 year ago
createTask
stopTask
リクエスト処理の実装なのでスクショはありません。
src/api/client/fetch/task.ts
src/pages/api
以下の処理を1つずつ実装してみました。
それぞれ、ソースコードにチェックしてほしい点をコメントします。 上記2点の実装が問題なさそうであれば、マージしてその他の API Routes の実装も行おうと思います。
Note 本 PR 作成当初、動的なパスパラメータを含むリクエスト処理 ( stopTask ) については API Rotues 上でも動的ルーティングの形を取っていました ( 8424587f402a232b7296c18492469ac2231b3ea1 , d6b71b89ed838e047c99a3e4f5385562073e47f0 )。 しかし、API Routes上での動的ルーティングを廃止し、クライアントコンポーネントから API Routes へのリクエストを送信する際のパラメータは全てリクエストボディに含めるようにしました ( dbab97a369c8d306f4c3a3902d1ac190b6e8549e ) 。 結果的に、動的パスパラメータの有無による差異はあまりなくなりました🙏
上記Noteのような方針を取った理由は以下のとおりです
req.query
src/api/server/fetch/task.ts
とくになし
The latest updates on your projects. Learn more about Vercel for Git ↗︎
issueURL
101
この PR で対応する範囲 / この PR で対応しない範囲
createTask
,stopTask
のエンドポイントを実装するStorybook の URL、 スクリーンショット
リクエスト処理の実装なのでスクショはありません。
変更点概要
src/api/client/fetch/task.ts
の実装 クライアントコンポーネントから呼び出す、API Routes 向けのリクエスト処理を実装しますsrc/pages/api
) の実装 フロントエンドからバックエンドに API リクエストを送信するための中継用エンドポイントを実装しますレビュアーに重点的にチェックして欲しい点
以下の処理を1つずつ実装してみました。
stopTask
)createTask
)それぞれ、ソースコードにチェックしてほしい点をコメントします。 上記2点の実装が問題なさそうであれば、マージしてその他の API Routes の実装も行おうと思います。
上記Noteのような方針を取った理由は以下のとおりです
req.query
)は全ての値が Optional で型定義する必要があり、処理が煩雑になること→ リクエストボディでパラメータを受け取り、dto を
src/api/server/fetch/task.ts
のメソッドに渡せば問題ないため補足情報
とくになし