bootcamp
bootcamp copied to clipboard
[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ラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
他issueがレビューまたは質問のご回答待ちになったので着手します。 よろしくお願いします。
@komagata 個人的に気になったところなのですが
通知部分を整理するのにactive_deliveryを使ったものに変えたいと思っているが、事前準備としてabstract_notifierに対応させる必要がある。
こちらに関して、active_deliveryのどこら辺の実装を見て、先にactive_notiferに対応させないとダメだとお気づきになったのでしょうか。まだまだその辺の理解が及ばず、質問させていただきました。
@yuki-snow1823 abstruct_notifierを知ったのはREADMEのRelated projectsに書いてあったからです。
active_deliveryはaction_mailer的な処理をまとめてやってくれるライブラリなので自前の処理をaction_mailerと同じインターフェースに書き換える必要があるなと考えていたらまさにabstruct_notifierがそのために用意されたライブラリだとわかり、見た瞬間に飛びついた感じです。
@komagata ご説明ありがとうございます。
自分もGemを開発するようなことがあればRelated Projectを示したいなと思いました。 そしてそういう応用のさせ方があると知れてよかったです!
@komagata こちらの本番環境での動作チェックは、実際にテスト用のQ&Aを作ってどなたかに回答をしてもらい 通知を確認するような流れで大丈夫でしょうか...?
@yuki-snow1823 はい、そちらでお願いします〜
動作確認完了しました!
遅れてすみません!完了しました。