@AntonKhorev 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])
Does this make any difference besides running the db query earlier? Or do we
want to run it earlier?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6338#pullrequestreview-3155822438
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6338/review/3155822...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev