commew / timelogger-web

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

タスクの計測開始・再開APIへのリクエストパラメータについて検討 #119

Closed kuniyuki-f closed 1 year ago

kuniyuki-f commented 1 year ago

完了の定義

補足情報

https://github.com/commew/timelogger-web/pull/115#discussion_r1320492081 のコメントでご指摘いただいていましたが、 @kuniyuki-f が見逃していたため別のIssueを立てました🙏

kuniyuki-f commented 1 year ago

@keitakn @stkzk3110 お疲れ様です!

下記の議論について、相談させていただきたいです🙏 お手隙でご確認いただき、ご意見いただければと思います!

議論

タスクの計測を開始・再開する時に、現在はフロントエンドからstartAtというリクエストパラメータを送信しております!

https://github.com/commew/timelogger-web/pull/115#discussion_r1320492081 で、以下のようなコメントをいただきました。

startAtはこれは、フロントエンドで指定した値をバックエンドでそのまま保存するという想定でしょうか? もしどうしてもそうしたいというのでなければ、不要じゃないかなと思います。 変更当時に見落していて申し訳ないのですが、タスク記録開始にもありますね。

個人的にはこちらの意見に賛成で、以下のようにコメントに返信しています。

自分も計測の開始時間みたいなアプリの状態についてはバックエンド側で管理する方が良いと思うので、startAtは不要かなと感じました!

みなさんのご意見も伺いたいです。 お手隙でご確認・コメントいただければ幸いです。 どうぞよろしくお願いいたします🙇

keitakn commented 1 year ago

@kuniyuki-f

内容確認しました!自分もスルーしてしまってすいませんでした!

確かにおっしゃる通りサーバー側でデータを保持したほうが安定なので startAt はなしの方向で大丈夫だと思います👍

kuniyuki-f commented 1 year ago

が完了したのでクローズします!