[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