fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

ブログ公開した際の通知が、WIPでも通知されてしまう。公開後もブログを編集すると通知されてしまう。 #7862

Closed machida closed 3 months ago

machida commented 4 months ago

FBCの公式ブログを公開すると、みんなに通知が飛ぶ。

というバグがある。

このように修正してください。 つまり、公開したとき一回だけ通知するようにしてください。 ブログ記事は publish_at という値があるので、それで公開前、公開後が判断できます。

motohiro-mm commented 4 months ago

@machida

お疲れ様です。 issueについて伺いたいことがあります。

ブログ公開した際の通知が、WIPでも通知されてしまう。

上記バグが再現できず、私が調べた範囲では現在WIPでは通知されていませんでした。 以下のコード内のwip?でWIPの場合には通知されないようになっていました。 https://github.com/fjordllc/bootcamp/blob/main/app/models/article_notifier.rb#L6

公開後もブログを編集すると通知されてしまう。

上記バグは、公開ボタンを押すたびに通知されていたので、再現できました。

いずれにしてもissueにあるように、

となるように修正する、という認識でよろしいでしょうか。 バグが再現出来ず不安だったので確認させていただきました。 お手隙の際に回答お願いいたします。

machida commented 4 months ago

@motohiro-mm 調査ありがとうございます!! WIPでは通知されない、了解ですー その認識でOKですー🙆‍♂️ では、そこはそのままで、「公開された記事をその後更新した際は通知をしない。」だけ対応をお願いします🙏

motohiro-mm commented 4 months ago

@machida 回答ありがとうございます!🙏承知しました!

motohiro-mm commented 3 months ago

@komagata お疲れ様です。 お手隙の際に本番環境での確認をお願いいたします!

確認手順は以下です。

  1. メンター権限をもつユーザーでログイン
  2. https://bootcamp.fjord.jp/articles/new(ブログ新規作成)よりブログを作成しWIPとして保存する
  3. 別ユーザーでログインし、通知が来ていないことを確認する
  4. WIP記事を書いたユーザーで再度ログインし、WIP記事を公開にする
  5. 別ユーザーでログインし、通知が来ていることを確認する
  6. 記事を公開したユーザーで再度ログインし、公開した記事を内容修正して更新する
  7. 別ユーザーでログインし、通知が来ていないことを確認する

煩雑な手順でお手数おかけいたします。 別ブラウザから別のユーザーでログインしておき、ブログの作成・公開のたびに通知の有無を随時確認していただくとログイン・ログアウトの手間を少なくできるかと思います。

よろしくお願いいたします!

komagata commented 3 months ago

@motohiro-mm 本番環境で確認できました〜!

motohiro-mm commented 3 months ago

@komagata

ありがとうございます〜!🙌

motohiro-mm commented 3 months ago

本番環境での確認をしていただきましたので、こちらのissueはcloseにします!