@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 %>

Happy to restore the "unhide" action, but I'm still a bit confused. When a user 
self-deletes, their data is scrubbed. How do admins know that the scrubbed 
users is the same?

Although... I'm seeing now that `email` is not scrubbed (`new_email` is). So 
I'm guessing that's how these accounts are identified? Do admins "unhide" these 
users, leaving them to log in and set their (hitherto scrubbed) username, etc 
themselves?

If the above is right, I would have thought this would be a GDPR issue, not 
scrubbing their email. I'm assuming this has been discussed and deemed 
acceptable?

Are you ok with me restoring only the "unhide" in user/list, leaving the action 
out of user/show?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6564#discussion_r2618830506
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

Reply via email to