[openstreetmap/openstreetmap-website] Trace import: remove activerecord-import gem (PR #5038)

Tom Hughes notifications at github.com
Sun Aug 4 13:54:58 UTC 2024


@tomhughes commented on this pull request.



>          tracepoints.each do |tp|
           tp.run_callbacks(:save) { false }
           tp.run_callbacks(:create) { false }
         end
 
-        Tracepoint.import!(tracepoints)
+        Tracepoint.insert_all!(tracepoints.map(&:attributes))

There are no validation on lat/lon at the moment so that's not surprising...

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

Message ID: <openstreetmap/openstreetmap-website/pull/5038/review/2217756571 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240804/53d9f70b/attachment.htm>


More information about the rails-dev mailing list