@nertc commented on this pull request.
> @@ -44,6 +44,15 @@ def index
end
@issues, @newer_issues_id, @older_issues_id = get_page_items(@issues,
:limit => @params[:limit])
+
+ @unique_reporters = @issues.each_with_object({}) do |issue, reporters|
+ user_ids = issue.reports.order(:created_at => :desc).map(&:user_id).uniq
+ reporters[issue.id] = {
+ :count => user_ids.size,
+ :users => User.where(:id =>
user_ids.first(3)).index_by(&:id).values_at(*user_ids.first(3))
Thanks for the suggestion. PR was updated. User selection was changed with much
smaller and cleaner code you suggested.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1933491750
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/4990/review/2580380...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
