[openstreetmap/openstreetmap-website] Do more preloading of tags and members (PR #6338)

mmd notifications at github.com
Thu Aug 21 16:07:42 UTC 2025


@mmd-osm commented on this pull request.



> @@ -1,7 +1,7 @@
 class RelationMembersController < ElementsController
   def show
     @type = "relation"
-    @feature = Relation.preload(:relation_members => :member).find(params[:id])
+    @feature = Relation.preload(:relation_members => { :member => :element_tag }).find(params[:id])

I think there's a typo here. It should be element_tags with "s" at the end. Right now, prefetching isn't really working.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6338/review/3141365342 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250821/dc191e77/attachment.htm>


More information about the rails-dev mailing list