[openstreetmap/openstreetmap-website] Account deletion cool-down period (PR #4313)
Anton Khorev
notifications at github.com
Tue Oct 24 18:36:30 UTC 2023
As I posted in the dwg chat:
- the delay is 1 day to make sure that there are no open changesets when users are deleted
- there's a setting, delays are disabled when set to 0
- I assumed that no possible law/regulation can exist that would require us to act faster than within a day
- no checks if the user is blocked, only the last changeset opening time
- I'd check the closing time, but there's no index together with uid
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4313
-- Commit Summary --
* Add user account deletion delay test helper
* Add earliest allowed deletion time to user model
* Disable delete account button if there are recent changesets
* Disallow account deletion after confirmation
-- File Changes --
M app/controllers/account/deletions_controller.rb (4)
M app/controllers/accounts_controller.rb (14)
M app/helpers/application_helper.rb (4)
M app/models/user.rb (8)
M app/views/account/deletions/show.html.erb (10)
M config/locales/en.yml (1)
M config/settings.yml (2)
A test/controllers/account/deletions_controller_test.rb (65)
M test/controllers/accounts_controller_test.rb (19)
M test/models/user_test.rb (38)
M test/test_helper.rb (11)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4313.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4313.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4313
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4313 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231024/0a637da6/attachment-0001.htm>
More information about the rails-dev
mailing list