[openstreetmap/openstreetmap-website] A new foundation for user notifications (PR #6837)

Pablo Brasero notifications at github.com
Wed Mar 11 14:06:23 UTC 2026


@pablobm commented on this pull request.



> +  def primary_and_foreign_key_types
+    config = Rails.configuration.generators
+    setting = config.options[config.orm][:primary_key_type]
+    primary_key_type = setting || :primary_key
+    foreign_key_type = setting || :bigint
+    [primary_key_type, foreign_key_type]
+  end

>From the generator indeed. Should I remove it then? I think we don't have any special config, so I can just remove the `:id => primary_key_type`, etc.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6837/review/3929855897 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260311/1cdf3ffa/attachment.htm>


More information about the rails-dev mailing list