お知らせにあるWIPの処理をnewspaperに置き換える

[akingo55]

Issue

• https://github.com/fjordllc/bootcamp/issues/5494

変更点

お知らせのWIPの処理をnewspaperに置き換えました。 お知らせの新規作成時、更新時にお知らせを公開するときの処理に影響があります。 見た目上、動作上の変更はありません。

確認方法

※ローカルにfeature/replace-announce-processing-from-callback-to-newspaperブランチを落として切り替えてください。

1. mentormentaroでログインする

2. お知らせを新規作成し、「全員」に「公開」し、お知らせが公開されていることを確認する

3. 一度ログアウトし、別のユーザーでログインする（今回はkomagataでログインしました）

4. 通知にさきほど公開したお知らせの通知が来ていることを確認する

5. bin/rails cで最新のお知らせを取得し、published_atに公開日時が入っていることを確認する

irb(main):004:0> Announcement.last
  Announcement Load (0.8ms)  SELECT "announcements".* FROM "announcements" ORDER BY "announcements"."id" DESC LIMIT $1  [["LIMIT", 1]]
=>
#<Announcement:0x000000010aba2538
 id: 1061722995,
 title: "おしらせテスト４",
 description: "ああああああああ",
 created_at: Sun, 02 Oct 2022 13:56:42.444950000 JST +09:00,
 updated_at: Sun, 02 Oct 2022 13:57:11.395091000 JST +09:00,
 user_id: 534981761,
 target: "all",
 wip: false,
 published_at: Sun, 02 Oct 2022 13:57:11.394889000 JST +09:00>
irb(main):005:0>

[akingo55]

@AyakaTakashima お疲れ様です！ 以前newspaperの置き換えのIssueを対応されていたかと思うので、こちらのPRをレビューしていただきたいです！:pray: お手隙によろしくお願いします！

[akingo55]

@AyakaTakashima ありがとうございます！

@komagata メンバーのレビューが通りましたので、お手隙にご確認お願いします🙏🏻

[akingo55]

@komagata コンフリクト解消しましたので、再度確認お願いします:pray:

[komagata]

@akingo55 すみません、またconflictが起こってしまったので修正をお願いします〜

[akingo55]

@komagata 遅くなりすみません:pray: 再度コンフリクト解消しましたので、ご確認お願いします！

[akingo55]

@komagata すみません、遅くなりました:pray: 修正完了しましたので、再度レビューお願いします。

[akingo55]

@komagata コンフリクト修正＆announcementの処理を1つにまとめました。（処理が散らばっておりわかりづらかったため） 開発環境にて以下の動作確認済みです。

• お知らせがwipの時は通知飛ばないこと（published_atはnullでwipがtrueであること）
• お知らせ公開された時は通知が飛ぶこと
• 公開時にはpublished_atに日時が入り, wipカラムがfalseに更新されること

テストはこちらのPRで追加されていたので、新規追加分はなさそうです。 https://github.com/fjordllc/bootcamp/pull/5662/files

再レビューお願いします。:pray:

[akingo55]

@komagata お手数ですが、こちらレビューお願いいたします。🙇‍♀️

[akingo55]

@komagata 確認お願いします！

[komagata]

@akingo55 マージコミットが含まれているようなので、rebaseの方式で、マージコミットができないようにお願いします〜

[akingo55]

@komagata すみません！マージコミット消しましたので再度確認お願いします！