[openstreetmap/openstreetmap-website] Fix naming in abstract adapter initializer (PR #6476)
Pablo Brasero
notifications at github.com
Mon Oct 27 13:57:13 UTC 2025
pablobm left a comment (openstreetmap/openstreetmap-website#6476)
Ooopsie, so I gather that this never worked? :cold_sweat:
Questions:
- What led to including this monkeypatch?
- Is this necessary any more? The original implementation looks ok to me (start at https://github.com/rails/rails/blob/dcb29df5b25006dd92f4fc10295f0177a33db93c/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb#L1193).
- If necessary, is the big conditional still required?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6476#issuecomment-3451424504
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6476/c3451424504 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251027/ef50ea86/attachment.htm>
More information about the rails-dev
mailing list