[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