[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