include rentals on rentals#index (#71)

BookingSync · Jul 3, 2017 · 73826d6 · 73826d6
1 parent 78cd1f0
commit 73826d6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 ### master
 
 * Update appraisal gemfiles
+* Fix N+1 query by including rentals on rentals#index
 * Speed up javascript search on rentals index page
 
 ### 0.9.0 - 2016-12-25

diff --git a/app/controllers/bookingsync_portal/admin/rentals_controller.rb b/app/controllers/bookingsync_portal/admin/rentals_controller.rb
@@ -9,7 +9,7 @@ def index
         @visible_rentals = current_account.rentals.visible
         @remote_accounts = current_account.remote_accounts
         @remote_rentals_by_account = current_account.remote_rentals.ordered
-          .includes(:remote_account).group_by(&:remote_account)
+          .includes(:remote_account, :rental).group_by(&:remote_account)
       end
 
       def show