Ensure user profiles and questions are eager loaded

Retrospring · Oct 22, 2023 · dd31a70 · dd31a70
1 parent 2432010
commit dd31a70
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 3 deletions.
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
@@ -8,7 +8,7 @@ class UserController < ApplicationController
   after_action :mark_notification_as_read, only: %i[show]
 
   def show
-    @pinned_answers = @user.answers.pinned.order(pinned_at: :desc).limit(10)
+    @pinned_answers = @user.answers.pinned.includes([{user: :profile}, :question]).order(pinned_at: :desc).limit(10)
     paginate_answers { |args| @user.cursored_answers(**args) }
 
     respond_to do |format|

diff --git a/app/models/answer/timeline_methods.rb b/app/models/answer/timeline_methods.rb
@@ -6,7 +6,7 @@ module Answer::TimelineMethods
   define_cursor_paginator :cursored_public_timeline, :public_timeline
 
   def public_timeline(current_user: nil)
-    joins(:user)
+    includes([{user: :profile}, :question])
       .then do |query|
         next query unless current_user
 

diff --git a/app/models/list/timeline_methods.rb b/app/models/list/timeline_methods.rb
@@ -5,9 +5,10 @@ module List::TimelineMethods
 
   define_cursor_paginator :cursored_timeline, :timeline
 
-  # @return [Array] the lists' timeline
+  # @return [ActiveRecord::Relation<Answer>] the lists' timeline
   def timeline(current_user: nil)
     Answer
+      .includes([{user: :profile}, :question])
       .then do |query|
         next query unless current_user
 

diff --git a/app/models/question/answer_methods.rb b/app/models/question/answer_methods.rb
@@ -7,6 +7,7 @@ module Question::AnswerMethods
 
   def ordered_answers(current_user: nil)
     answers
+      .includes([{user: :profile}, :question])
       .then do |query|
         next query unless current_user