Merge pull request #2 from alexandruclapou/hotfix/fixed-np1-queries

n+1 queries problem fixed
alexandruclapou · Feb 25, 2022 · a25fc77 · a25fc77
2 parents d14c485 + 558ccec
commit a25fc77
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -230,6 +230,7 @@ GEM
 
 PLATFORMS
   x86_64-darwin-20
+  x86_64-darwin-21
 
 DEPENDENCIES
   bootsnap (>= 1.4.4)

diff --git a/app/controllers/api/scores_controller.rb b/app/controllers/api/scores_controller.rb
@@ -5,7 +5,7 @@ class ScoresController < ApplicationController
     before_action :validate_score_user_id, only: :destroy
 
     def user_feed
-      scores = Score.all.order(played_at: :desc, id: :desc)
+      scores = Score.all.order(played_at: :desc, id: :desc).includes(:user)
       serialized_scores = scores.map(&:serialize)
 
       response = {