Avoid N+1s when listing subscription line items in admin

solidusio · Jan 3, 2023 · b1da743 · b1da743
1 parent 17da0ea
commit b1da743
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/views/spree/admin/subscriptions/index.html.erb b/app/views/spree/admin/subscriptions/index.html.erb
@@ -137,7 +137,7 @@
           <td><%= subscription.interval.inspect %></td>
           <td><%= render 'state_pill', subscription: subscription %></td>
           <td><%= render 'processing_state_pill', subscription: subscription %></td>
-          <td><%= subscription.line_items.map { |line_item| line_item.subscribable&.name }.join(", ") %></td>
+          <td><%= subscription.line_items.includes(subscribable: :product).map { |line_item| line_item.subscribable&.name }.join(", ") %></td>
           <td class="actions">
             <% if subscription.state_events.include?(:cancel) %>
               <%=