From 56df8050dff5ac33ec52efb838229abd03ca073d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Wed, 28 Nov 2018 13:11:53 +0100
Subject: [PATCH] Use `@cards.each` instead of `@cards.find_each`

Using `find_each` ignores the scope order we set in `Widget::Card.body`,
and since we don't expect to have thousands of cards, using batches
isn't necessary.

This way we remove the "WARN Scoped order and limit are ignored, it's
forced to be batch order and batch size" message we were getting in the
specs.
---
 app/views/welcome/_cards.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/welcome/_cards.html.erb b/app/views/welcome/_cards.html.erb
index d579e6e75..8a42521c4 100644
--- a/app/views/welcome/_cards.html.erb
+++ b/app/views/welcome/_cards.html.erb
@@ -1,7 +1,7 @@
 <h3 class="title"><%= t("welcome.cards.title") %></h3>
 
 <div class="row" data-equalizer data-equalizer-on="medium">
-  <% @cards.find_each do |card| %>
+  <% @cards.each do |card| %>
     <%= render "card", card: card %>
   <% end %>
 </div>