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

Gregory Igelmund notifications at github.com
Mon Oct 23 11:05:09 UTC 2023


@grekko commented on this pull request.



> +    <thead>
+      <tr>
+        <th><%= t ".muted_user" %></th>
+        <th class="d-flex justify-content-end"><%= t ".actions" %></th>
+      </tr>
+    </thead>
+    <tbody>
+      <% @muted_users.each do |user| %>
+        <tr>
+          <td>
+            <%= user_thumbnail_tiny user %>
+            <%= link_to user.display_name, user_path(user) %>
+          </td>
+          <td class="text-nowrap text-end">
+            <%= link_to t(".unmute"), unmute_user_path(user), :method => :delete, :class => "btn btn-sm btn-primary" %>
+            <%= link_to t(".send_message"), new_message_path(user), :class => "btn btn-sm btn-secondary" %>

> An option to send a message to a muted user seems a bit strange when by definition any reply will wind up in the muted messages and won't be notified?

I can imagine situations in which I muted another user just to prevent them from spamming my email inbox but still want to keep the conversation open.

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

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


More information about the rails-dev mailing list