[openstreetmap/openstreetmap-website] Move html_safe declaration for user_mailer from layout to helper (PR #5358)

Andy Allan notifications at github.com
Wed Nov 27 18:19:56 UTC 2024


This allows us to enable output safety checks for all code within erb files.

`rubocop --auto-gen-config` ignores code within the erb files, so it is easier to maintain an exclusion on the helper than inside the layout.

You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5358

-- Commit Summary --

  * Move html_safe declaration for user_mailer from layout to helper

-- File Changes --

    M .erb_lint.yml (2)
    M .rubocop_todo.yml (1)
    M app/helpers/user_mailer_helper.rb (2)
    M app/views/layouts/user_mailer.html.erb (2)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5358.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5358.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/5358 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241127/8cc497e6/attachment.htm>


More information about the rails-dev mailing list