@nertc commented on this pull request.


> @@ -60,6 +61,13 @@
           <td class="text-nowrap"><%= link_to t(".reports_count", :count => 
issue.reports_count), issue %></td>
           <td><%= link_to reportable_title(issue.reportable), 
reportable_url(issue.reportable) %></td>
           <td><%= link_to issue.reported_user.display_name, 
issue.reported_user if issue.reported_user %></td>
+          <td>
+            <% unique_reporters = issue.reports.uniq { |report| report.user.id 
} %>

To avoid Ruby on Rails logic code in the HTML, `preload` logic was moved in the 
controller. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1812585358
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/4990/review/2388381...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to