ブログ（articles）が公開されたら、管理者、メンター、現役一般受講生、現役研修生に通知する #7615
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hirano-vm4
force-pushed
the
feature/article-publication-notifications
branch
from
d23619d
to
9eb1e36
Compare
hirano-vm4
force-pushed
the
feature/article-publication-notifications
branch
from
ea10f83
to
28ecdd0
Compare
hirano-vm4
force-pushed
the
feature/article-publication-notifications
branch
from
28ecdd0
to
ff33b8f
Compare
|
お疲れ様です🙏 急ぎではありませんので、無理ないタイミングで全く問題ありません！(1~2週間後とかでもOKです) ご検討のほど、よろしくお願いいたします🙇♂️ |
|
お忙しい中すいません🙏こちらいかがでしょうか？ |
|
@hirano-vm4 |
a-terumoto-gs
approved these changes
Apr 12, 2024
|
お忙しん中、確認いただきありがとうございました🙏引き続きよろしくお願いします！ こちらチームメンバーによるレビューが終わりました！お忙しいところ、恐れ入りますがレビューをお願いいたします🙏 |
komagata
approved these changes
Apr 24, 2024
| @@ -0,0 +1,8 @@ | |||
| # frozen_string_literal: true | |||
|
|
|||
| class ArticleNotificationDestroyer | |||
| @@ -170,4 +170,12 @@ def no_correct_answer | |||
|
|
|||
| ActivityMailer.with(question:, receiver:).no_correct_answer | |||
| end | |||
|
|
|||
| def create_article | |||
There was a problem hiding this comment.
mailのpreviewもしっかり追加しているのナイスです～
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
概要
ブログ（articles）が公開された際、管理者、メンター、現役一般受講生、現役研修生にメール・サイト内通知が飛ぶ機能を実装するRPです。
変更確認方法
feature/article-publication-notificationsをローカルに取り込むbin/setupを実行foreman start -f Procfile.devでサーバーを立ち上げるkomagataアカウントでログインし http://localhost:3000/articles/new にアクセステスト等入力し、ページ下部にある「公開する」を押して保存が完了するまで待つkomagataアカウントのまま、投稿者には通知がされないことをサイト内通知で確認（ページ右上の通知ボタンを押して未読部分を確認してください）machida（管理者）mentormentaro（メンター）kensyu(研修生)kimura（受講生）でそれぞれログインしてサイト内通知がきていること・通知からブログ詳細に飛べることを確認komagataアカウントでログインし、 http://localhost:3000/articles のブログ一覧に行って今回投稿したブログの詳細に入り、ページ下部にある「削除」を押して消すmachida（管理者）mentormentaro（メンター）kensyu(研修生)kimura（受講生）いずれかでログインして通知を開き、サイト内通知が削除されていることを確認レビュー参考資料
通知のトリガーからの流れを把握しやすくなるように図を作成したので参考資料として添付します。
active_delivery
active_delivery
active_deliveryで通知をまとめる - komagataのブログ
Newspaper
komagata/newspaper: Provides a pub/sub mechanism for ruby.
newspaperでActiveRecordのCallbacksを置き換える - komagataのブログ
Newspaperを読んでみた
Railsのメール機能
Action Mailer の基礎 - Railsガイド
Screenshot
変更前
新機能のためなし
変更後
ブログ（article）の投稿ページ下部の「公開する」を押して保存
通知を確認rails/mailers/activity_mailer/create_article のプレビュー/letter_opene