bootcamp
bootcamp copied to clipboard
fjordllc
ブラウザのバックボタンで戻った時にユーザー登録ボタンの状態を更新する
Issue
- #7553
概要
DOMContentLoadedイベントでチェックボックスの状態を元に「参加する」ボタンの有効無効を切り替えていたが、ブラウザのバックボタンで戻った場合、DOMContentLoadedイベント時にはまだチェックボックスの状態が復元されておらず、結果、チェックボックスの状態と「参加する」ボタンの状態が合わせられていなかった。
※ブラウザによって挙動が異なっていたため、違いの調査については上記Issueのコメントに記載しています。
- https://github.com/fjordllc/bootcamp/issues/7553#issuecomment-2008631911
変更確認方法
-
bug/fix-submit-button-on-browser-backをローカルに取り込む -
localhost:3000/users/newにアクセスする -
カード番号を入力する- ダミーの番号として、
4242 4242 4242 4242が使用可能（有効期限は任意の将来の日付、セキュリティコードは任意の3桁の数字）
- ダミーの番号として、
- 他の入力必須項目を未入力のまま、以下のチェックボックスにチェックを入れ、
参加するボタンをクリックする-
アンチハラスメントポリシーに同意 -
利用規約に同意
-
- 未入力項目があるため、未入力エラーが出る。その画面で以下の項目を確認する
-
アンチハラスメントポリシーに同意のチェックボックスがチェック状態のままであること -
利用規約に同意のチェックボックスがチェック状態のままであること -
参加するボタンが有効（クリックできる状態）であること
-
- ブラウザのバックボタンで戻る
- 以下の項目を確認する
-
アンチハラスメントポリシーに同意のチェックボックスがチェック状態のままであること -
利用規約に同意のチェックボックスがチェック状態のままであること -
参加するボタンが有効（クリックできる状態）であること
-
Screenshot
変更前
変更後
@kurumadaisuke お疲れ様です。 こちらレビューをお願いできますでしょうか🙏（全く急ぎではありません） もしご都合悪いなどありましたら、その旨、仰っていただけたらと思います。 よろしくお願い致します。🙇♂️
@naokinaokiboo ご連絡ありがとうございます🙏 ちょっと仕事がバタバタしていてFBCに時間があまり割けないのですが、 お時間いただいてもいいのであればレビュー承ります🙇♂️ もし、急ぎ等の理由があればお手数ですが他の方でもよろしいでしょうか🙇♂️？
@kurumadaisuke
ちょっと仕事がバタバタしていてFBCに時間があまり割けないのですが、 お時間いただいてもいいのであればレビュー承ります🙇♂️
はい。急ぎではないので、大丈夫です。 お忙しいところ恐縮ですが、よろしくお願い致します。🙇♂️
@naokinaokiboo
確認が遅くなりすいません。
動作確認でlocalhost:3000/users/newにアクセスして、対象のチェックボックスにチェックを入れ
そこから「ブラウザのバックボタンで戻る」の次はもう一度localhost:3000/users/newにアクセスをする。であっています？🙏
どういった挙動を再現すれば分かっておらずで🙏
@kurumadaisuke
そこから「ブラウザのバックボタンで戻る」の次はもう一度localhost:3000/users/newにアクセスをする。であっています？🙏
返信遅くなり申し訳ありません。🙏
確認方法の手順にカード番号の入力の手順が抜けていました。申し訳ありません。（この手順が抜けていたため、混乱を招いてしまったと思います）🙇♂️
確認方法を一部更新しましたので、お手数ですが、再度ご確認お願い致します。
ブラウザのバックボタンで戻った時の挙動を確認したいため、バックボタンで戻った後、再度localhost:3000/users/newへのアクセスは不要です。