[openstreetmap/openstreetmap-website] Make account deletion remove the email address when possible (PR #3488)

Tom Hughes notifications at github.com
Mon May 23 18:30:41 UTC 2022


@tomhughes commented on this pull request.



> @@ -21,6 +21,9 @@
 <ul>
   <li><%= t ".delete_profile" %></li>
   <li><%= t ".delete_display_name" %></li>
+  <% unless current_user.changesets.exists? -%>

I've rebased this on the current master and introduced the `must_retain_email?` method to centralise the logic around when it should be retained as suggested.

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

Message ID: <openstreetmap/openstreetmap-website/pull/3488/review/982198707 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220523/5a875e2c/attachment.htm>


More information about the rails-dev mailing list