<p></p>
<p><b>@tomhughes</b> requested changes on this pull request.</p>

<p dir="auto">Although this is described as showing the "last three unique reporters" there isn't actually any ordering that I can see, and ordering on time is hard to combine with uniqification...</p><hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1679721999">app/views/issues/index.html.erb</a>:</p>
<pre style='color:#555'>> @@ -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 } %>
</pre>
<p dir="auto">This could probably be better done as <code class="notranslate">issue.reports.preload(:user).map(&:user).uniq</code> which would also get a list of users instead of a list of reports.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1679726947">app/views/issues/index.html.erb</a>:</p>
<pre style='color:#555'>> @@ -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 } %>
+            <% unique_reporters[..2].each_with_index do |report, index| %>
</pre>
<p dir="auto">This could be done more simply with something like:</p>
<pre class="notranslate"><code class="notranslate">safe_join(unique_reporters[..2].map { |u| link_to u.display_name, u }, ",")
</code></pre>
<p dir="auto">That's assuming the previous edit to make the reporter list a list of users.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1679727854">app/views/issues/index.html.erb</a>:</p>
<pre style='color:#555'>> @@ -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 } %>
+            <% unique_reporters[..2].each_with_index do |report, index| %>
+              <%= link_to(report.user.display_name, report.user) %><%= "," unless index == 2 || index == unique_reporters.size - 1 %>
+            <% end %>
+            <%= "..." if unique_reporters.size > 3 %>
</pre>
<p dir="auto">Maybe instead of truncating the list here we should put the whole thing in but use CSS to limit the width and add an ellipsis? With a tooltip to show the follow value even?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4990#pullrequestreview-2180762354">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPVTMEDVBKN5MWPBYLZMVDYBAVCNFSM6AAAAABK4MEPN6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCOBQG43DEMZVGQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLI6QXFU5WBPCKGCOKTZMVDYBA5CNFSM6AAAAABK4MEPN6WGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUB7PFPE.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/4990/review/2180762354</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/4990#pullrequestreview-2180762354",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4990#pullrequestreview-2180762354",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>