Skip to content

Conversation

@antkryt
Copy link
Contributor

@antkryt antkryt commented Apr 24, 2025

Purpose

fix broken users

Changes

  • do not unregister users when deactivate spam

QA Notes

How I was able to recreate it:
In admin go to managment commands -> Ban spam users by regular expression and ban some users (Actually there may be other ways to recreate it using spam feature in different ways like management commands, scheduled spam checks and so on). They will become unregistered, disabled and confirmed. Then we can reactivate account in admin to make it enabled but not registered

Documentation

Side Effects

Ticket

https://openscience.atlassian.net/browse/ENG-7704

Copy link
Contributor

@Johnetordoff Johnetordoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@brianjgeiger brianjgeiger changed the base branch from feature/pbs-25-08 to feature/pbs-25-09 May 2, 2025 13:31
@brianjgeiger brianjgeiger merged commit d12a1c2 into CenterForOpenScience:feature/pbs-25-09 May 2, 2025
6 checks passed
ihorsokhanexoft pushed a commit to ihorsokhanexoft/osf.io that referenced this pull request Jun 4, 2025
## Purpose
fix broken users

## Changes
- do not unregister users when deactivate spam

## QA Notes
How I was able to recreate it:
In admin go to managment commands -> Ban spam users by regular expression and ban some users (Actually there may be other ways to recreate it using spam feature in different ways like management commands, scheduled spam checks and so on). They will become unregistered, disabled and confirmed. Then we can reactivate account in admin to make it enabled but not registered

## Ticket
https://openscience.atlassian.net/browse/ENG-7704
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.

3 participants