Skip to content

HSB-497 chore: user deletetion if not solo team owner #4508

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

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

mirarifhasan
Copy link
Contributor

Closes HSB-497

What's changed

This PR updated the user deletion. Previously if a user a team owner, that user can not be deleted.
Now, a user can be deleted if he is not solo team owner in any of teams.

Notes to reviewers

3 ways a user can be deleted.

  1. User can delete himself
  2. User can delete by admin
  3. User can delete via API

@mirarifhasan mirarifhasan self-assigned this Nov 3, 2024
@RaHehl
Copy link
Contributor

RaHehl commented Nov 5, 2024

Would it be possible to control whether the workspace is deleted automatically when the last owner is removed, instead of blocking the deletion, through a configuration variable? This could make the behavior more flexible depending on the use case.

@RaHehl
Copy link
Contributor

RaHehl commented Nov 5, 2024

Or perhaps even add a force_delete option for both the API and GUI to handle automatic workspace deletion when the last owner is removed?

@AndrewBastin
Copy link
Member

@RaHehl that could be something we can tackle down the line. We are still a bit skeptical about having this in the GUI, but this could be something we can add in the API as an initial start.

@AndrewBastin AndrewBastin force-pushed the feat/user-delete-condition branch from 3db0b02 to ef48f67 Compare November 5, 2024 15:32
@AndrewBastin AndrewBastin merged commit d3999b9 into patch Nov 5, 2024
1 check passed
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