commew / timelogger-web

時間記録アプリ
https://timmew.commew.net
MIT License
2 stars 0 forks source link

#101 API Routes から参照するAPIリクエスト処理の配置を修正した #102

Closed kuniyuki-f closed 1 year ago

kuniyuki-f commented 1 year ago

issueURL

101

この PR で対応する範囲 / この PR で対応しない範囲

Storybook の URL、 スクリーンショット

ロジックの修正なのでスクショはありません。

変更点概要

src/api/client/fetch/task.ts で定義していた処理・テストコードをまるっと src/api/server/fetch/task.ts に移動しました。

レビュアーに重点的にチェックして欲しい点

基本的に appToken が必要なリクエスト処理ばかりだったので、全部 API Routes を経由して呼ばれる想定です。

小さな PR にはなりますが、 #101 の作業の土台になるリファクタリングなので念のため client から server に移動すること自体問題なさそうかご確認頂きたいです。

とくに新規実装や処理内容を変えた、ということはないので、ファイルの配置が変わったことだけ確認いただければと思います🙏

補足情報

とくになし。

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
timelogger-web ✅ Ready (Inspect) Visit Preview Aug 1, 2023 5:33am