@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 %>
Do you mean self-deletion (which scrubs personal data), or "hiding" by an admin
(which leaves data intact)?
If actually talking about self-deletion, I didn't know that was a use case.
When is that desired?
If talking about "hiding", I was going with admins manually suspending all
hidden users directly on the DB, as per the PR description:
> Note that we'll be left with pre-existing "hidden" users. After discussing
> with @Firefishy, these users can be moved to the "suspended" state by the Ops
> team with a script, but it's not a priority.
You did raise a question about the feasibility of this at
https://github.com/openstreetmap/openstreetmap-website/issues/6546#issuecomment-3586714352,
but then accepted it as non-blocking, if I understood correctly.
If however you'd still like to be able to do this manually, I can bring back
the "unhide". Perhaps only for the user list, to reduce the number of options
in user/show?
Or am I missing something?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6564#discussion_r2611051965
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
