@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/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
