bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

[abstract_notifier] 回答通知を置き換えたい

Open komagata opened this issue 2 years ago • 4 comments

経緯

通知部分を整理するのにactive_deliveryを使ったものに変えたいと思っているが、前準備としてabstract_notifierに対応させる必要がある。

やること

回答通知の実装をabstract_notifierに置き換えたい。

既に導入したPRがあるのでそちらを参考にする。

komagata avatar Apr 23 '22 17:04 komagata

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

github-actions[bot] avatar Jun 30 '22 01:06 github-actions[bot]

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

github-actions[bot] avatar Aug 31 '22 01:08 github-actions[bot]

他issueがレビューまたは質問のご回答待ちになったので着手します。 よろしくお願いします。

yuki-snow1823 avatar Sep 02 '22 02:09 yuki-snow1823

@komagata 個人的に気になったところなのですが

通知部分を整理するのにactive_deliveryを使ったものに変えたいと思っているが、事前準備としてabstract_notifierに対応させる必要がある。

こちらに関して、active_deliveryのどこら辺の実装を見て、先にactive_notiferに対応させないとダメだとお気づきになったのでしょうか。まだまだその辺の理解が及ばず、質問させていただきました。

yuki-snow1823 avatar Sep 02 '22 02:09 yuki-snow1823

@yuki-snow1823 abstruct_notifierを知ったのはREADMEのRelated projectsに書いてあったからです。

active_deliveryはaction_mailer的な処理をまとめてやってくれるライブラリなので自前の処理をaction_mailerと同じインターフェースに書き換える必要があるなと考えていたらまさにabstruct_notifierがそのために用意されたライブラリだとわかり、見た瞬間に飛びついた感じです。

komagata avatar Sep 05 '22 11:09 komagata

@komagata ご説明ありがとうございます。

自分もGemを開発するようなことがあればRelated Projectを示したいなと思いました。 そしてそういう応用のさせ方があると知れてよかったです!

yuki-snow1823 avatar Sep 09 '22 00:09 yuki-snow1823

@komagata こちらの本番環境での動作チェックは、実際にテスト用のQ&Aを作ってどなたかに回答をしてもらい 通知を確認するような流れで大丈夫でしょうか...?

yuki-snow1823 avatar Sep 16 '22 06:09 yuki-snow1823

@yuki-snow1823 はい、そちらでお願いします〜

komagata avatar Sep 18 '22 08:09 komagata

スクリーンショット 2022-09-21 8 32 17

動作確認完了しました!

yuki-snow1823 avatar Sep 20 '22 23:09 yuki-snow1823

@yuki-snow1823 下記の通りIssueのCloseをお願いします〜

チーム開発の作業手順 | FBC

komagata avatar Sep 22 '22 05:09 komagata

遅れてすみません!完了しました。

yuki-snow1823 avatar Sep 26 '22 01:09 yuki-snow1823