[openstreetmap/openstreetmap-website] Add standard List-XXX headers to some notification mails (PR #4534)

Tom Hughes notifications at github.com
Sat Feb 24 11:53:01 UTC 2024


This adds a subscribe/unsubscribe for changeset comments that mirrors the recently added one for diary comments and adjusts the emails for changeset comments, diary comments and notes to add appropriate List-XXX headers to them.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Rename path for changeset API subscription methods
  * Move mechanics of changeset subscriptions into the model
  * Add unsubscribe link to changeset notification mails
  * Add standard List-XXX headers to some notification messages

-- File Changes --

    M app/abilities/ability.rb (1)
    M app/controllers/api/changesets_controller.rb (10)
    M app/controllers/changesets_controller.rb (29)
    M app/mailers/user_mailer.rb (30)
    M app/models/changeset.rb (12)
    M app/views/browse/changeset.html.erb (4)
    A app/views/changesets/no_such_entry.html.erb (5)
    M app/views/user_mailer/changeset_comment_notification.html.erb (2)
    M app/views/user_mailer/changeset_comment_notification.text.erb (2)
    M config/locales/en.yml (20)
    M config/routes.rb (6)
    M test/controllers/api/changesets_controller_test.rb (20)
    M test/controllers/changesets_controller_test.rb (133)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/4534 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240224/707dfae0/attachment-0001.htm>


More information about the rails-dev mailing list