[openstreetmap/openstreetmap-website] Show "reporting user" on "issues" screen (PR #4990)
David Tsiklauri
notifications at github.com
Fri Jan 24 14:41:58 UTC 2025
@nertc commented on this pull request.
> @@ -44,6 +44,11 @@ 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|
+ reporters[issue.id] = issue.reports.order(:created_at => :desc).preload(:user).map(&:user).uniq
PR was updated. Now "loading users" only affects first 3 users that should be shown.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4990#discussion_r1928787293
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4990/review/2572830635 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250124/e54c22ec/attachment.htm>
More information about the rails-dev
mailing list