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

[Gregory Igelmund]

@grekko commented on this pull request.



> @@ -14,5 +14,8 @@
     <li class="nav-item">
       <%= link_to t(".oauth2_authorizations"), oauth_authorized_applications_path, :class => "nav-link #{'active' if controller_name == 'oauth2_authorized_applications'}" %>
     </li>
+    <li class="nav-item">
+      <%= link_to t(".muted_users"), user_mutes_path, :class => "nav-link #{'active' if controller_name == 'user_mutes'}" %>

yup. I changed `messages/outbox.html.erb` to also reuse `messages/_messages_table.html.erb` and therefore pass a custom `inner_partial` and a new configuration – `columns` to `messages/_messages_table.html.erb`.

I'd understand if the view logic seems a bit overcomplicated now, but I think its still not too hard to read and therefore the message table logic is kept together in a small set of files.

See c0cbf1c28da68739bc854eb8249f5298ce16e8a5

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

Message ID: <openstreetmap/openstreetmap-website/pull/4284/review/1756417476 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231129/badb8a01/attachment-0001.htm>