<p></p>
<p><b>@tomhughes</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4313#discussion_r1370823206">app/views/account/deletions/show.html.erb</a>:</p>
<pre style='color:#555'>> @@ -31,5 +31,13 @@
   <li><%= t ".retain_email" %></li>
 </ul>
 
-<%= link_to t(".delete_account"), account_path, { :method => :delete, :class => "btn btn-danger", :data => { :confirm => t(".confirm_delete") } } %>
+<% if @allowed_at %>
+  <div class="alert alert-warning">
+    <%= t ".recent_editing_html", :time => friendly_date_time_element(@allowed_at) %>
+  </div>
+  <button class="btn btn-danger" disabled><%= t(".delete_account") %></button>
</pre>
<p dir="auto">I wonder if we should just leave the button out when deletion is not possible? Even a disabled "danger" button still looks very clickable - the difference to a non-disabled one is quite subtle.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4313#discussion_r1370828310">config/settings.yml</a>:</p>
<pre style='color:#555'>> @@ -53,6 +53,8 @@ api_timeout: 300
 web_timeout: 30
 # Periods (in hours) which are allowed for user blocks
 user_block_periods: [0, 1, 3, 6, 12, 24, 48, 96, 168, 336, 731, 4383, 8766, 87660]
+# Account deletion cooldown period since last changeset close (in hours)
+user_account_deletion_delay: 24
</pre>
<p dir="auto">Do we want to have a default for this? or should it default to off but be something we can enable operationally? In other words should we assume that other users of this code will want this.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4313#discussion_r1370831327">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -256,6 +256,7 @@ en:
         retain_notes: Your map notes and note comments, if any, will be retained but hidden from view.
         retain_changeset_discussions: Your changeset discussions, if any, will be retained.
         retain_email: Your email address will be retained.
+        recent_editing_html: "Due to your recent editing activity you'll be able to delete your account in %{time}"
</pre>
<p dir="auto">This doesn't read very well to me but I'm struggling to come up with a better phrasing, at least as a single sentence. The best I've come up with is:</p>
<blockquote>
<p dir="auto">As you have edited recently your account cannot currently be deleted.<br>
Deletion will be possible in %{time}.</p>
</blockquote>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4313#pullrequestreview-1695904483">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKJVUQWBPDOEIWZFR3YBAW6XAVCNFSM6AAAAAA6OENTSGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMOJVHEYDINBYGM">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNQ35ZMCLQFEU4MNDLYBAW6XA5CNFSM6AAAAAA6OENTSGWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTTFCVZOG.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/4313/review/1695904483</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/4313#pullrequestreview-1695904483",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4313#pullrequestreview-1695904483",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>