[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