commew / timelogger-web

時間記録アプリ

https://timmew.commew.net

MIT License

2 stars 0 forks source link

計測中のタスクを /timer で表示できるようにした #96

Closed kuniyuki-f closed 1 year ago

kuniyuki-f commented 1 year ago

issueURL

59

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

ページアクセス時に、クライアントコンポーネント側で計測中のタスクを取得し、一覧に表示させる処理を実装する
新規タスク作成時や、その他タスクのステータス変更処理が行われた際の画面更新処理は別PRで対応する
taskCategoryId を用いて API から実際のカテゴリーグループ名、カテゴリー名を取得する処理は別PRで対応する

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

デザイン
計測中タスクがある場合のテンプレート https://63d52217f1430a5ad69846cd-veruhijxsd.chromatic.com/?path=/story/templates-timertemplate--default
計測中タスクがない場合のテンプレート https://63d52217f1430a5ad69846cd-veruhijxsd.chromatic.com/?path=/story/templates-timertemplate--no-tasks-recording

変更点概要

レンダリングのロジックとは直接関係ありませんが、features/taskに定義している、関数インターフェースFetchTasksRecordingの返り値の型をTasksからTaskRecording[]に修正しています。
pages/timer.tsxからtemplates/TimerTemplate.tsxを呼び出す形で実装しています。
- TimerTemplate.tsx では最低限のデザインコーディングも行っています。
custom hooksのuseTasks.tsを実装しています。

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

コード内にコメントしています！

補足情報

とくになし

vercel[bot] commented 1 year ago

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

Name	Status	Preview	Comments	Updated (UTC)
timelogger-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2023 8:26am