feat:親子タスクdueDateのバリデーション

Ebi-web / react-room

React部屋で作るTodoアプリケーション

react-room-lime.vercel.app

Other

0 stars 0 forks source link

feat:親子タスクdueDateのバリデーション #36

Closed Ebi-web closed 2 years ago

Ebi-web commented 2 years ago

親子タスク間の締め切り日のバリデーションを追加したものです

日付だけを見て2つのdayjsインスタンスの前後関係を判定する関数をhelpersディレクトリに追加しました
~バリデーションルールとしては「あるタスクのdueDateは親タスクのdueDateと同じかそれ以降でなくなてはならない」「あるタスクのdueDateは子タスクのdueDateと同じかそれ以前でなくてはならない」を実装しています~
バリデーションルールとしては「あるタスクのdueDateは親タスクのdueDateと同じかそれ以前でなくなてはならない」「あるタスクのdueDateは子タスクのdueDateと同じかそれ以降でなくてはならない」を実装しています
現在のタスク状態をstateではなくlocalstorageから取得するように統一しました

close #33

vercel[bot] commented 2 years ago

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

Name	Status	Preview	Updated
react-room	✅ Ready (Inspect)	Visit Preview	Jul 31, 2022 at 7:34AM (UTC)

masa5555 commented 2 years ago

すみません、今更気づいたのですが、

　あるタスクのdueDateは親タスクのdueDateと同じかそれ以降でなくてはならない

これって正しいのでしょうか？なぜなら、僕のイメージとしては、

親タスク：論文を書く 8/20
- 小タスク：構成を箇条書きする 8/1
- 小タスク： 背景の章を書く 8/5

というようにタスクを作るイメージだからです。えびさわさんのイメージと認識が違うのであれば、共有お願いします。

Ebi-web commented 2 years ago

すみません、今更気づいたのですが、

あるタスクのdueDateは親タスクのdueDateと同じかそれ以降でなくてはならない

これって正しいのでしょうか？なぜなら、僕のイメージとしては、
親タスク：論文を書く 8/20
- 小タスク：構成を箇条書きする 8/1
- 小タスク： 背景の章を書く 8/5
というようにタスクを作るイメージだからです。えびさわさんのイメージと認識が違うのであれば、共有お願いします。

あああ確かに...そうですね、ありがとうございますちゃっと直します

masa5555 commented 2 years ago

早速ありがとうございます。いいと思います。