fjordllc / fjord-books_app

fjord-books_app/app/controllers/users_controller.rb

Lines 4 to 6 in c762120

    
           def index 
        
             @users = User.order(id: :desc).page(params[:page]) 
        
           end

view側で user.avatar を表示する処理をしているため、ActiveStorageでN+1が発生しているようです。
.with_attached_avatar で解決するようですが、プラクティスとして扱うためにちょっと変わったブランチ構成になっている状態なので、PullRequestを投げる前に一旦issueとして報告します。
指摘のあった日報: https://bootcamp.fjord.jp/reports/23268

次の2点を確認できれば進められそうだなと思います。

そもそも修正する方針でOKでしょうか？
修正する場合は、07-follow_users ~ 99-completed の各ブランチに対してPullRequestを出す方針で問題ないでしょうか？

@betachelsea 報告ありがとうございます！
あー、たしかに今のコードだとN+1になりますね・・・。
お手本のコードとしてはイマイチなので、PRを出してもらうと助かります！ 🙇

そもそも修正する方針でOKでしょうか？
修正する場合は、07-follow_users ~ 99-completed の各ブランチに対してPullRequestを出す方針で問題ないでしょうか？

どちらもyesです。

https://fjordbootcamp.esa.io/posts/81 （要ログイン）もあわせて参考にしてみてください〜。

@JunichiIto 参考esaありがとうございます！週末になりそうですが、PullRequest出してみます 👍 しばらくお待ちください！

まだ各branchへのmergeが終わっていませんが(push権限不足)、 07-follow_users への変更は適用されたのでcloseします。

	def index
	@users = User.order(id: :desc).page(params[:page])
	end

N+1問題が出ている