fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

休会時のお知らせメール内に復帰予定日と理由を表示するようにした #8144

Open Ryooo-k opened 1 month ago

Ryooo-k commented 1 month ago

Issue

概要

休会時のお知らせメール内に「復帰予定日」と「休会理由」が載っていないため、メールを開いた時点で「復帰予定日」と「休会理由」を表示するようにしました。

変更確認方法

  1. chore/add-return-date-and-reason-to-suspension-notification をローカルに取り込む i. git fetch origin pull/8144/head:chore/add-return-date-and-reason-to-suspension-notification ii. git checkout chore/add-return-date-and-reason-to-suspension-notification
  2. foreman start -f Procfile.dev でローカルサーバーを立ち上げる
  3. ユーザー名kimura、パスワードtesttestでログインする
  4. 休会手続きにアクセスする
  5. 復帰予定日と休会理由(自由記入)を入力する。
  6. 「休会についての注意を読みましたか?」の下にあるチェックボックスを選択し「休会する」ボタンを押す
  7. letter_opener_webにアクセスする
  8. メール本文内に「復帰予定日」と「休会理由」が表示されているか確認する

Screenshot

変更前

スクリーンショット 2024-10-22 14 45 05

変更後

スクリーンショット 2024-10-22 14 43 04

出力フォーマットについて

退会時のお知らせメールでは、退会理由が表示されるようになっています。今回の休会時のお知らせメールは、退会時のお知らせメールと同じ出力形式にしています。

参照先:app/views/activity_mailer/retired.html.slim

スクリーンショット 2024-10-22 15 01 45
Ryooo-k commented 1 month ago

@thmz337 お疲れ様です! こちらのレビューをお願いしたいのですが可能でしょうか?

thmz337 commented 1 month ago

@Ryooo-k

承知しました!いつ頃を目処にすれば良いでしょうか?

Ryooo-k commented 1 month ago

@thmz337 ありがとうございます! 急ぎではないので、お任せいたします👍

Ryooo-k commented 2 days ago

@thmz337 ご確認ありがとうございました!

@komagata メンバーレビューが完了しました! レビューをお願いいたします。