bootcamp
bootcamp copied to clipboard
fjordllc
[abstract_notifier] 研修生の日報の更新の通知の実装を置き換えたい
経緯
通知部分を整理するのにactive_deliveryを使ったものに変えたいと思っているが、前準備としてabstract_notifierに対応させる必要がある。
やること
研修生の日報の更新の通知の実装をabstract_notifierに置き換えたい。
既に導入したPRがあるのでそちらを参考にする。
- abstract_notifierで通知を実装する - komagataのブログ
- 現状の研修生の日報の更新の通知の実装
- abstract_notifierを導入 by komagata · Pull Request #4673 · fjordllc/bootcamp
- palkan/active_delivery: Ruby framework for keeping all types of notifications (mailers, push notifications, whatever) in one place
- palkan/abstract_notifier: ActionMailer-like interface for any type of notifications
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
@machida issueの内容について質問になります🙏 「研修生の日報の更新の通知」とありますが、この「通知」とは右上の通知マークへの通知のことでしょうか？現在の状況を確認してみたところ、右上の「通知」へは何も通知が来ません。通知の定義をしっかり理解しておきたいと思い質問させていただきました。よろしくお願いします。
-
[email protected]でログインし日報を更新する。
-
[email protected]でログインし確認する。
@shirotamaki お答えしますー 研修生が日報を書くと、研修生の所属している企業のアドバイザーに通知が飛ぶ仕組みが用意されています。 今回の通知は、企業のアドバイザーが受け取る通知になります。
@machida 度々すいません💦 研修生が所属している企業のアドバイザーへの通知ですが、新規で日報を作成した際の通知は確認できました。 「日報の更新」の定義について確認なのですが、既存の日報の内容を修正して更新した場合は「日報の更新」に該当せず、通知は飛ばないという理解で合っていますでしょうか？
@komagata @machida お疲れさまです！こちらのissueの動作確認を本番環境でお願いできますでしょうか？
確認方法
- 研修生のアカウントでログインする
- 日報を作成し提出する
- 所属企業のアドバイザーのアカウントでログインし通知が来ていることを確認する
- 所属企業のアドバイザー宛にメールが届いていることを確認する