日報が確認OKされていない場合、メンターがコメントした時にアラートが出るように実装

[su-su-su-su]

元々メンターが日報にコメントをだけ入れて確認済みにしないまま画面を離れようとしたらアラートを出す。 というイシューだったのですが、ブラウザの仕様によりbeforeunloadイベントではカスタムメッセージを表示することができないことがわかりました。 そのことを町田さんに相談させていただいて、確認済になっていないとき、Bではなく、Aをクリックしたら、 「日報を確認済みにしていませんがよろしいですか？」 とアラートを出るようにしたいということで進めました。

Issue

• 6928

概要

日報が確認OKされていない場合、メンターがコメントした時にアラートが出るように実装しました。

変更確認方法

1. feature/alert-for-unconfirmed-daily-report-commentをローカルに取り込む
2. foreman start -f Procfile.devでローカル環境を立ち上げる
3. machidaでログイン
4. 未チェックの日報をクリック
5. コメントを書いてコメントするをクリック
6. 「日報を確認済みにしていませんがよろしいですか？」とアラートが出ることを確認

   Screenshot

変更前

変更後

念のため、以下の２つも確認しました。 メンター以外がコメントしてもアラートが出ないことも確認しました。

メンターが日報が以外にコメントしてもアラートが出ないことを確認しました。

[su-su-su-su]

@Judeeeee お疲れ様です。 可能でしたらお手隙の際に、こちらのPRのレビューをお願いできますでしょうか？ ご都合が合わない場合はおっしゃってください🙇‍♂️ どうぞよろしくお願いいたします!

[Judeeeee]

@su-su-su-su お疲れ様です〜！承知しました🙆‍♀️ できるだけ早くレビューするつもりですが、1週間目処で考えていただけると幸いです🙏

[su-su-su-su]

@Judeeeee お疲れ様です！ありがとうございます🙇‍♂️ 全然急ぎでないので、お手隙の際にレビュー頂ければと思います。 よろしくお願い致します🙏

[su-su-su-su]

@Judeeeee お疲れ様です！ レビューありがとうございます！

su-su-su-suさんが確認したことを客観的に示すためにも、テストを追加いただけると助かります🙇 私も過去にほぼ同じようなissueを担当しており、その時のPRが参考になるかもです〜！

アドバイスありがとうございます🙇‍♂️ Judeeeeeさんがされていた部分的に追加できるテストがなかったので、新規でテストを追加しました。

手隙の際に、再度ご確認頂けますでしょうか？ どうぞよろしくお願い致します🙏

[Judeeeee]

@su-su-su-su 修正ありがとうございます🙏 承知いたしました！ レビュー終わり次第また追って連絡させていただきますので、引き続きよろしくお願いいたします〜！

[su-su-su-su]

@Judeeeee

お疲れ様です。ご確認ありがとうございます🙇‍♂️

私の環境だと、テストを実行した際に落ちてしまうのですがsu-su-su-suさんの環境では問題なくパスしているでしょうか？🤔 何度実行しても以下のエラーが出てしまっている状況です😢

私の方で再度テストをしました。

test/system/comments_test.rb をテストしました。

bin/rails test test/system/comments_test.rb                                                    
Running via Spring preloader in process 15261
Run options: --seed 8377

# Running:

................................
[Minitest::CI] Generating test report in JUnit XML format...

Finished in 105.349211s, 0.3038 runs/s, 0.7024 assertions/s.
32 runs, 74 assertions, 0 failures, 0 errors, 0 skips

test/system/comments_test.rb のhow_confirm_dialog_if_report_is_not_confirmedのみテストをしました。

bin/rails test test/system/comments_test.rb -n /show_confirm_dialog_if_report_is_not_confirmed/

Running via Spring preloader in process 25194
Run options: -n /show_confirm_dialog_if_report_is_not_confirmed/ --seed 45360

# Running:

.
[Minitest::CI] Generating test report in JUnit XML format...

Finished in 21.066426s, 0.0475 runs/s, 0.0949 assertions/s.
1 runs, 2 assertions, 0 failures, 0 errors, 0 skips

となっていて私の方ではテストは通っていたのですが、テストの方法が違っていたりしているのでしょうか？

[Judeeeee]

@su-su-su-su お待たせしました💦 ご確認ありがとうございます🙏

結論、私の方でもテストが通ることを確認しました🙇 私のローカル環境の問題でして、bin/setupを実行し再度テストしたところ問題が解消されました。 お手数をおかけしました🙇

[su-su-su-su]

@Judeeeee

私のローカル環境の問題でして、bin/setupを実行し再度テストしたところ問題が解消されました。

ご確認ありがとうございます🙏 自分もそうなったときbin/setupをしてみます。 参考になりました！ありがとうございます🙇‍♂️

[su-su-su-su]

@komagata お疲れ様です。 approveいただきましたのでレビューをお願いできますでしょうか 。 よろしくお願いいたします。

[su-su-su-su]

@komagata

お疲れ様です。 ご指摘頂いた点を修正いたしましたのでお手隙の際にご確認お願いします。

[su-su-su-su]

@komagata お疲れ様です。 ご指摘頂いた点を修正いたしましたのでお手隙の際に再度ご確認お願いします。

[komagata]

@su-su-su-su 本番環境で確認しました。

[su-su-su-su]

@komagata ご確認ありがとうございました。