search

fjordllc / bootcamp

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

日報でメンション付きのコメントが付くと通知が2件届く #6676

Closed JunichiIto closed 1 year ago

JunichiIto commented 1 year ago

タイトルの通りです。どちらか1件だけだとありがたいです。

fbc2 fbc

ちなみに過去に同じようなissueをいくつも出してます。。

monyatto commented 1 year ago

@JunichiIto cc: @komagata @machida お疲れさまです！ ローカル環境から以下の手順でバグの再現をしようとしましたが、確認ができませんでした。

IDhajimeで日報を作成。 IDmachidaで上記の日報をwatch IDhajimeで@machidaメンションをしてコメント

image image

4709 と同様に#4546 にて既に解決済みかと思ったのですがいかがでしょうか。

現在もこちらのバグが発生しているか、あるいは再現手順に間違いがありましたらご指摘いただけると幸いです。

ご確認よろしくお願いいたします🙏

JunichiIto commented 1 year ago

@monyatto ご確認ありがとうございます。

https://github.com/fjordllc/bootcamp/issues/4709 と同様にhttps://github.com/fjordllc/bootcamp/pull/4546 にて既に解決済みかと思ったのですがいかがでしょうか。

4546 のリリースが4/30で、僕の報告が6/27なので、時系列を見ればまだ発生していると考える方が自然かなと思います。

ローカル環境から以下の手順でバグの再現をしようとしましたが、確認ができませんでした。 現在もこちらのバグが発生しているか、あるいは再現手順に間違いがありましたらご指摘いただけると幸いです。

これは何とも言えないですね。。 僕はメールを受信した側なので、再現手順がわかるとすればコメントを送った生徒さんになると思います。 （と言ってもご本人は「ふつうにコメントしただけです」って言いそうですが）

なんとなくの予想ですが、この不具合は「毎回発生するとは限らない」とか「本番環境でしか発生しない」というような、再現性が高くない類いの不具合ではないかと推測します。 可能であれば6/27 18:30前後の本番環境のログを眺めるのが原因追及の近道なのかなーと思いました。

monyatto commented 1 year ago

@JunichiIto コメントありがとうございます！　開発MTGで相談してみます🙇🏻

JunichiIto commented 1 year ago

FYI

現在もこちらのバグが発生しているか

今日も発生していました。

fbc

https://bootcamp.fjord.jp/reports/76417#comment_136942

なぜ2分空いてるのかはわかりませんが。。

monyatto commented 1 year ago

【memo】 こちら7/12の開発MTGにて、本番環境で確認できるのが駒形さんのみということなので一旦対応保留。

machida commented 1 year ago

📝

ということなので、原因の特定が難しく、本番環境のログを見る必要があるため、一旦 @komagata さんをアサインし、ログを貼ってもらって原因のヒントを得てから着手することにします。

komagata commented 1 year ago

@monyatto (CC: @machida )これってなんのログを貼れば良いですかね？

必要であればこれが確認できるためのログ出力のコードを追加し、リリースして確かめるというてもあるかもしれません。

machida commented 1 year ago

📝 https://github.com/komagata/newspaper/pull/1

伊藤さんの出してくれたPRで解決しそう。 これを本番に反映して、直ったかどうかを確認する。

JunichiIto commented 1 year ago

伊藤さんの出してくれたPRで解決しそう。

うーん、どうですかね〜？ まったく同じメールが2件なら可能性が高いですが、そうではないので。

あと、to_prepareが複数回呼ばれるのは開発環境特有の事象ではないかなと想像しています。

もちろん、やってみないとわからないですが、after_initializeに切り替えたあともしばらく様子を見る必要がありそうです。

machida commented 1 year ago

@JunichiIto ありがとうございます！しばらく様子を見るようにします。

github-actions[bot] commented 1 year ago

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

github-actions[bot] commented 1 year ago

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。