[openstreetmap/openstreetmap-website] Interface to configure notification preferences (PR #7001)

[Pablo Brasero]

@pablobm commented on this pull request.



> +  # Required by ActionView in order to accept this in form_for.
+  # Normally it would provide an id to tell which record is
+  # targeted by the form. However our form targets
+  # the preferences of `current_user` specifically, not just
+  # any arbitrary record, so we can put any value here as it
+  # will be ignored.
+  def to_key
+    nil
+  end

Yeah, I was going back and forth while trying to remember how to make ActionView happy. Removed this in favour of the include now 👍 

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

Message ID: <openstreetmap/openstreetmap-website/pull/7001/review/4120229022 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260416/061cf80c/attachment.htm>