fjordllc
メール内リンクのメール通知オフをクリックした後に確認画面を追加
Issue
- #7837
概要
メール下部にある「メールでの通知オフはこちらから」をクリックした後に確認画面を追加しました。
- 修正前 クリック後すぐにメール通知はオフになり、トップページにリダイレクトして「メール配信を停止しました。」とメッセージが表示される。
- 修正後 クリック後、「メール通知をオフにしますか？」という確認画面が表示される。
変更確認方法
-
feature/confirm_to_cancel_mail_notificationsをローカルに取り込む -
foreman start -f Procfile.devでサーバーを立ち上げる -
hajimeでログインする（誰でもOKです） -
日報作成で本文に
@kimuraを入れて日報を提出する（メンションしてメール通知をとばす） - ログアウトし
kimuraでログインする - LetterOpenerでメールを開く
- メンション通知メール内の「メールでの通知オフはこちらから」をクリック
- 以下を順に確認する
- [ ] 「メール通知をオフにしますか？」という確認画面が表示されること
- [ ] 「オフにする」リンクをクリックするとトップページに遷移し、「メール配信を停止しました。」というメッセージが表示されること
- [ ] 「登録情報変更」でメール通知がオフになっていること
※ブランチを変更した後に通知されたメールでないと修正が確認できませんので注意してください。
Screenshot
変更前
https://github.com/fjordllc/bootcamp/assets/104712009/9f4dffb3-bac5-4048-9139-f8ead6397eae
変更後
https://github.com/fjordllc/bootcamp/assets/104712009/0f064f86-5f1a-4b21-bffa-5b85ff1e3915
@machida
お疲れさまです！ メール通知オフの確認画面のデザインをお願いいたします。 現状は以下のようになっています。
-
未ログイン
-
ログイン中
なにか不足がありましたらおっしゃってください🙏 よろしくお願いいたします！
@motohiro-mm お待たせしました:bowing_man: デザインを入れました！！
ログインしている、していないに関わらず同じ画面にしたいと思います。
最新のmainを取り込んだので、
git pull --rebase origin feature/confirm_to_cancel_mail_notifications
をお願いします🙏
@machida
お忙しいところデザインをいれていただきありがとうございました！🙌
確認させていただきます！
pull --rebaseの件も承知しました！
@Judeeeee
おつかれさまです！ お忙しいところ恐縮ですが、こちらのPRのレビューをお願いいたします🙏 ご都合が合わなければ遠慮なくおっしゃってください🙇♀️ 急ぎではありません！ よろしくお願いいたします！
@motohiro-mm お疲れ様です〜！承知です！ なるべく早く対応しますが、1週間目処にレビューさせていただきますね！ よろしくお願いいたします🙇
@Judeeeee
レビューいただきありがとうございました！ 修正とコメントをしましたので、再度ご確認をお願いいたします！ （コミットのプッシュをまちがったために、レビューへの返信がJudeeeeeさんのコメント部への返信になってしまいました🙇♀️見づらいですが遡ってご確認ください🙏）