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

Tom Hughes notifications at github.com
Tue Aug 26 17:29:46 UTC 2025


@tomhughes commented on this pull request.



> @@ -30,7 +30,7 @@ def index
 
     # Dump the details on a node given in params[:id]
     def show
-      @node = Node.find(params[:id])
+      @node = Node.includes(:element_tags).find(params[:id])
 

Yes strictly speaking that one probably doesn't make any real difference. I really just did it for consistency with the other method.

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

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


More information about the rails-dev mailing list