Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
概要
日報の学習日入力部分において、通常あり得ないほど過去の年月日が選択され日報が作られてしまうのを防ぐために、範囲バリデーションを作成しました。
2013年1月1日より前の日時を選択して提出ボタンを押しても、HTMLとモデルの双方のバリデーションでDBへの登録が防止されます。
変更確認方法
HTMLバリデーション確認後、モデルバリデーションを確認する流れです。
feature/add-validation-to-date-in-report
をローカルに取り込むgit fetch origin pull/8181/head:feature/add-validation-to-date-in-report
(2度目以降は上記ブランチのローカル変更に気をつけながら
--force
をつけてください)git switch feature/add-validation-to-date-in-report
foreman start -f Procfile.dev
でローカルサーバーを立ち上げる2000/01/01
のような2013年1月1日より前の日時を入力しますopt
+cmd
+i
)学習日
のinput
を見つけてくださいmin="2013-01-01"
部分をダブルクリックしますmin="2013-01-01"
を消去してエンターを押してください2000/01/01
のような2013年1月1日より前の日時を入力します学習日は2013年01月01日から今日以前の間の日付にしてください
)Screenshot
変更前
変更前はバリデーションがないため、そのまま保存されます。
変更後
基本的にはHTMLバリデーションが表示されます。
デベロッパーツールなどを使用し、HTMLバリデーションを無効化した場合のみ、モデルバリデーションが表示されます。