bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

ベストアンサーの通知をabstract_notifierに置き換えた

Open Nabegon opened this issue 2 years ago • 3 comments

Issue

  • #4683

概要

ベストアンサーの通知をabstract_notifierに置き換えました。仕様は変更していないため、変更前/変更後のスクリーンショットはありません。

確認方法

  1. feature/replace-the-best-answer-notification-with-abstract_notifierをローカルに取り込む
  2. bin/setup実行
  3. bin/rails sでサーバーを立ち上げる
  4. sotugyouでログインし、質問ページにいき、質問を作成する
  5. kensyuでログインし直し、sotugyouが作成した質問ページにいき、回答を投稿する
  6. sotugyouでログインし直し、kensyuの回答をベストアンサーにする
  7. kensyuでログインし直し、下記のようにkensyuの回答がベストアンサーに選ばれた通知を確認する image
  8. 通知一覧のページにいき、下記のようにkensyuの回答がベストアンサーに選ばれた通知を確認する image
  9. http://127.0.0.1:3000/letter_opener/にアクセスし、メールが送信されていることを確認する image

Nabegon avatar Aug 09 '22 20:08 Nabegon

@ai-24 さん、お疲れ様です!

こちらのPRのレビューをお願いしたいのですが、ご都合いかがでしょうか。急ぎではありません。 以前に@ai-24 さんも初回日報通知(サイト内通知)をabstract_notifierに置き換えるIssueに対応されていたため、レビューをお願いしたいなと思いました🙏

ご検討をお願いします🙇

Nabegon avatar Aug 24 '22 21:08 Nabegon

@ai-24 さん、お疲れ様です! レビューいただきありがとうございました!レビューいただいた内容を反映しましたので、お手数をおかけしますが再度ご確認のほどよろしくお願いします🙏

Nabegon avatar Aug 28 '22 21:08 Nabegon

@ai-24 さん、レビューありがとうございました!

@komagata さん、お疲れ様です!チームレビュー通りましたので、レビューお願いします🙏

Nabegon avatar Sep 01 '22 07:09 Nabegon

@komagata さん、お疲れ様です!コンフリクトを取り込む際にブランチを壊してしまい、再度ブランチを作り直しました。お手数ですが、こちらのPRの確認をお願いします🙏

https://github.com/fjordllc/bootcamp/pull/5522

よろしくお願いします。

Nabegon avatar Sep 14 '22 20:09 Nabegon

@Nabegon こちらのブランチはCloseで大丈夫でしょうか?でしたらブランチの削除もお願いします〜!

komagata avatar Sep 18 '22 07:09 komagata