[openstreetmap/openstreetmap-website] Account deletion cool-down period (PR #4313)

[Andy Allan]

@gravitystorm commented on this pull request.



> @@ -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 (in hours) since last changeset close; -24 to disable, 0 to make sure there aren't any open changesets when the deletion happens
+user_account_deletion_delay: -24

https://github.com/gravitystorm/openstreetmap-website/commit/7802efc8c986bf0648dccd8f1f80cca631cf3420 - I think the clearest thing is to have two configuration options - one to turn cooldown on and off, and the other to configure the length of the cooldown.

What do you both think? I prefer the separate boolean config, rather than having one option with mixed integer/nil (where 0 and nil will have significantly different behaviours).

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

Message ID: <openstreetmap/openstreetmap-website/pull/4313/review/1744558045 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231122/fa6f3cbb/attachment.htm>