[openstreetmap/openstreetmap-website] Rework UI to hide/delete/suspend users (PR #6564)

Pablo Brasero notifications at github.com
Tue Dec 16 09:49:33 UTC 2025


@pablobm commented on this pull request.



> -            <% if @user.may_unhide? %>
-              <li>
-                <%= link_to t(".unhide_user"), user_status_path(@user, :event => "unhide"), :method => :put, :data => { :confirm => t(".confirm") } %>
-              </li>
-            <% end %>

Well, hopefully not "horrible" as no lives or data were lost :slightly_smiling_face: 

I did ask twice about moving this to users/lists (see earlier in this thread). The logic is:
- The intent of this PR is to clarify and unclutter the admin options in users/show.
- When a user asks to revert a deletion, I can only guess that admin find them via users/lists, searching by email and potentially filtering by status=Deleted.

Therefore users/lists might be a better way to place this. Of course this is based in assumptions, and I'll be happy to move this back to users/show.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6564/review/3582229865 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251216/d50648e7/attachment.htm>


More information about the rails-dev mailing list