commew / timelogger-web

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

フロントエンドからNextApi経由でタスク計測再開をリクエストする処理を実装した #129

Closed kuniyuki-f closed 1 year ago

kuniyuki-f commented 1 year ago

issueURL

117

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

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

https://63d52217f1430a5ad69846cd-qkmhelrtyy.chromatic.com/?path=/story/components-completetaskbutton-completetask--default

変更点概要

128 での実装に引き続き、クライアントコンポーネント側からNextApi経由でタスク計測再開をリクエストする処理を実装し、実際にStartTaskButtonコンポーネントに組み込みました。

また、Complet, Stop の処理を実装した時に漏れていたクライアント側のテストコードも実装しました。 Issueの範囲から外れないように、とりあえず Start に関するテストコードのみを実装しています。

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

ソースコード内に記載します。

補足情報

とくになし

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 Sep 16, 2023 3:04am
kuniyuki-f commented 1 year ago

@keitakn

レビューありがとうございます!

タイトルに Draft : が入っていますが、PRはOpenになっていたので確認しました!

あああ😱 すいません、タイトルの修正を失念していました! 臨機応変にレビューいただいてありがとうございます🙏

タイトルからDraft:を削除したうえでマージしました!