bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

開発環境でコードを修正するとNewspaperに登録した処理が2回走る

Open JunichiIto opened this issue 1 year ago • 6 comments

概要

開発環境でコードを修正するとNewspaperに登録した処理が2回走る。

再現手順

https://bootcamp.fjord.jp/reports/77011 を参照。

期待される振る舞い

開発環境でコードを修正しても同じ処理が重複して走らない。

スクリーンショット

https://bootcamp.fjord.jp/reports/77011 を参照。

環境

(サーバーサイドの問題なので省略)

修正方法

config/initializers/newspaper.rbto_prepareafter_initializeに変更すると直りそうです。

https://railsguides.jp/autoloading_and_reloading_constants.html#%E3%82%A2%E3%83%97%E3%83%AA%E3%82%B1%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%E8%B5%B7%E5%8B%95%E6%99%82%E3%81%AE%E8%87%AA%E5%8B%95%E8%AA%AD%E3%81%BF%E8%BE%BC%E3%81%BF

JunichiIto avatar Jul 18 '23 22:07 JunichiIto

@komagata Newspaper側のREADMEを修正するPRも出しておきました。 https://github.com/komagata/newspaper/pull/1

JunichiIto avatar Jul 18 '23 22:07 JunichiIto

@JunichiIto README修正のPRありがとうございます。

komagata avatar Jul 31 '23 04:07 komagata

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

github-actions[bot] avatar Sep 30 '23 01:09 github-actions[bot]

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

github-actions[bot] avatar Oct 07 '23 01:10 github-actions[bot]

修正されていないので reopen します。

cafedomancer avatar Feb 02 '24 12:02 cafedomancer

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

github-actions[bot] avatar Apr 03 '24 01:04 github-actions[bot]

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

github-actions[bot] avatar Apr 10 '24 01:04 github-actions[bot]

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

github-actions[bot] avatar Jun 10 '24 01:06 github-actions[bot]

@komagata CC @machida これ、まだ修正されていないようですが、修正予定はありますか? https://github.com/fjordllc/bootcamp/blob/main/config/initializers/newspaper.rb

JunichiIto avatar Jun 10 '24 01:06 JunichiIto

@JunichiIto 修正したいと思っております。ですがまだ解決方法はわかっていないです。

komagata avatar Jun 12 '24 12:06 komagata

@komagata description に「config/initializers/newspaper.rbのto_prepareをafter_initializeに変更すると直りそうです。」って書いたんですが、これはダメそうですか?

JunichiIto avatar Jun 13 '24 09:06 JunichiIto

@JunichiIto すみません、伊藤さんのREADMEのPRでafter_initializeへの変更が済んだと勘違いしていました。 こちら試してみます!

komagata avatar Jun 26 '24 12:06 komagata

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

github-actions[bot] avatar Aug 26 '24 01:08 github-actions[bot]

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

github-actions[bot] avatar Sep 02 '24 01:09 github-actions[bot]