[openstreetmap/openstreetmap-website] Redirect to inbox after marking a message as read/unread (PR #4752)

Anton Khorev notifications at github.com
Sun May 5 12:47:13 UTC 2024


Fixes #4750 by redirecting when "Mark as unread" is clicked on a message page. For example Github notifications work like that: if you mark a notification as unread it redirects you to the notifications list.

I rename the `unread_message` variable in tests because the message doesn't stay unread.

I reuse the xhr test lines for referer test because xhr is not in use since the Turbo update.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Rename unread_message to message in messages controller test
  * Redirect to inbox after marking a message as read/unread disregarding referer

-- File Changes --

    M app/controllers/messages_controller.rb (2)
    M test/controllers/messages_controller_test.rb (66)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/4752 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240505/256fb269/attachment.htm>


More information about the rails-dev mailing list