[openstreetmap/openstreetmap-website] Do more preloading of tags and members (PR #6338)
mmd
notifications at github.com
Thu Aug 21 16:25:13 UTC 2025
@mmd-osm commented on this pull request.
> @@ -2,7 +2,7 @@ class OldRelationMembersController < OldElementsController
def show
@type = "relation"
@current_feature = Relation.find(params[:id])
- @feature = OldRelation.preload(:old_members => :member).find([params[:id], params[:version]])
+ @feature = OldRelation.preload(:old_members => { :member => :old_tags }).find([params[:id], params[:version]])
Prefetching isn't working here as well. In my branch, I used `element_tags` instead of `old_tags`. Members of a relation point to the current nodes/ways/relations, even for OldRelations.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6338#pullrequestreview-3141429602
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6338/review/3141429602 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250821/4ca86291/attachment.htm>
More information about the rails-dev
mailing list