[openstreetmap/openstreetmap-website] Add UserMute to control message visibility (PR #4284)

Gregory Igelmund notifications at github.com
Wed Dec 6 21:50:20 UTC 2023


@gravitystorm thanks for the review comments! I think I've addressed them all.

I just noticed and addressed 1 bug and 1 odd thing:
- The `Mute this User`-action on `app/views/users/show.html.erb` was displayed to unauthenticated users. I think this was confusion.
- The redirects in `UserMutes#destroy` after Unmuting the last muted user caused a redirect loop – not anymore.

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

Message ID: <openstreetmap/openstreetmap-website/pull/4284/c1843745813 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231206/7f317678/attachment.htm>


More information about the rails-dev mailing list