[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