bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

新規入会者の通知にroleも表示してほしい

Open machida opened this issue 2 years ago • 4 comments

貼り付けた画像_2022_10_03_17_24

  • 一般受講生の場合
    • 今のまま
  • アドバイザーの場合
    • 🎉 XXXさん(アドバイザー)が新しく入会しました!
  • メンターの場合
    • 🎉 XXXさん(メンター)が新しく入会しました!
  • 管理者の場合
    • 🎉 XXXさん(管理者)が新しく入会しました!
  • 研修生の場合
    • 🎉 XXXさん(研修生)が新しく入会しました!

machida avatar Oct 03 '22 08:10 machida

@machida これ良いですね!

komagata avatar Oct 04 '22 12:10 komagata

📝

通知の仕組みを理解する

  • [x] Notification.create! を行うとユーザが画面を開いた時に新規通知が追加されている。prefix: 🎉
  • [x] notification_mailermail to を行うとメールでの通知がいく。prefix: [FBC]

どうやって通知が送られているか

  • [x] メール→ action_mailermail to メソッド
    • [x] user が退会、もしくはメールの設定をオフにしていたらメール送信しない。
  • [x] 画面での通知 → Notification.create or Notification.newnotification メソッドで作成している。
  • [x] notify_now ですぐに通知している?

通知の文言はどこで決まるか

  • [x] 「さんが新しく入会しました!」で検索する
  • [x] notification_mailer.signed_up ← こちらはメール
  • [x] activity_notifier.signed_up ← 今回はこっち。通知のみ。

新規入会者の通知をローカルで出すにはどうするか

  • [x] ~Notification.new すれば出てきそうな気がする。試してみる。~ User が必要だったりいろんな依存関係があったりしそう。メールのメソッドまで呼べなさそう。
  • [x] 実際に新規入会者をフォームから作成する
    • [x] ロールを持たないユーザはサインアップから作成できる
    • [x] ロールを持つユーザの作成。 admin を持つユーザでログイン(komagata, machida)→ 管理メニュー>管理ページ>企業 を開く→アドバイザーや研修生のリンクからサインアップ
    • [x] 上記でユーザを作成したら、admin のユーザでログインした画面を開く。通知を見ることができる。 CleanShot 2022-10-23 at 15 28 18

ロールはどうやって特定するか

  • どうやら複数のロールをもてるみたい。メンターとアドバイザーを持っている人がいる。
  • 画像がロールかな CleanShot 2022-10-23 at 14 22 40
  • ここにもロールがある CleanShot 2022-10-23 at 15 10 45
  • User の student? だったら早期リターンするメソッド作ると良さそう。
  • そのあとはロールによって配列にプラスしていく。
  • そして最後に ()で括ってあげると文言が作成されそう
  • もしくは User に直接 roles_strings 作るかどうか。ちょい微妙か
  • user_decorator に staff_roles という同じようなメソッドがあるのでそれを参考にする CleanShot 2022-10-23 at 14 30 59@2x

yamatatsu10969 avatar Oct 22 '22 06:10 yamatatsu10969

@machida こちら調査したところ、通知はメールでの件名の通知とフィヨルドの画面での通知の2種類があることがわかりました。 メールの方にもroleを追加した方がよろしいでしょうか? ご確認よろしくお願いいたします!

yamatatsu10969 avatar Oct 22 '22 06:10 yamatatsu10969

📝 新規入会者がいた場合、メールと fjord の画面に通知が飛んでくる

メール

CleanShot 2022-10-23 at 15 34 58

通知

CleanShot 2022-10-23 at 15 24 38

yamatatsu10969 avatar Oct 23 '22 06:10 yamatatsu10969

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] avatar Dec 23 '22 01:12 github-actions[bot]

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

github-actions[bot] avatar Dec 31 '22 01:12 github-actions[bot]