Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show edit history of deleted users #4224

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AntonKhorev
Copy link
Contributor

Links to previous/next changesets from /changeset/<id>. Also links to /user/<name>/history, which becomes available for everyone. <name> is whatever the deletion procedure wrote to user's display name. This name is only used in urls, everywhere else the user is shown as "deleted".

Implements #1311

@tomhughes
Copy link
Member

I think whether we can/should do this depends on us reaching a conclusion on how we can/should handle deleted accounts in general which in turn means deciding what, from a data protection point of view, we are allowed to do.

@AntonKhorev
Copy link
Contributor Author

All that's made available here on the web interface is already available on the api.

@tomhughes
Copy link
Member

Yes why we need to figure out the correct policy and apply it consistently everywhere - that might this PR or it might mean making the API match the web interface.

@AntonKhorev
Copy link
Contributor Author

All changesets are available in dumps too.

@AntonKhorev
Copy link
Contributor Author

Your current policy:
6c1d73a#diff-44438ce218f5287c58d0017f965d888715635d94280669896f75841fbd7b4cd7R245

All you can change is to make the edit history available only for registered users. But then that would likely apply to all history pages, not just ones of deleted users.

@AntonKhorev AntonKhorev force-pushed the deleted-user-history branch 2 times, most recently from 9470b2d to b7f882e Compare March 19, 2024 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants