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

[Project Settings] Edge case: brand new owner can remove themselves #3114

Open
imnasnainaec opened this issue May 3, 2024 · 1 comment · May be fixed by #3194
Open

[Project Settings] Edge case: brand new owner can remove themselves #3114

imnasnainaec opened this issue May 3, 2024 · 1 comment · May be fixed by #3194
Assignees
Labels
bug Something isn't working lowpriority This might not be worked on project Size: S Small sized issue

Comments

@imnasnainaec
Copy link
Collaborator

A project administrator (e.g., Simon) can remove themselves from a project.
A project owner (e.g., Garfunkel) can transfer project ownership to a project administrator (e.g., Simon).

If admin Simon opens project settings, Users tab; then on a different computer owner Garfunkel transfers project ownership to Simon; then new owner Simon who still sees himself as just admin Simon removes himself from the project without refreshing the page, he is successful. Result: Garfunkel is a project admin and the project has no owner.

This is a ridiculous edge case, but it shows there could be some safeguard added in the backend to prevent project owner removal without replacement.

@imnasnainaec imnasnainaec added bug Something isn't working project lowpriority This might not be worked on labels May 3, 2024
@imnasnainaec imnasnainaec added this to To do in Combine Kanban via automation May 3, 2024
@imnasnainaec imnasnainaec moved this from To do to Discuss in planning meeting in Combine Kanban May 3, 2024
@imnasnainaec imnasnainaec added the Size: S Small sized issue label Jun 5, 2024
@jasonleenaylor
Copy link
Contributor

Probably we should just throw an error from the backend if we remove an owner without replacing it.

@imnasnainaec imnasnainaec self-assigned this Jun 28, 2024
@imnasnainaec imnasnainaec linked a pull request Jun 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lowpriority This might not be worked on project Size: S Small sized issue
Projects
No open projects
Combine Kanban
Discuss in planning meeting
Development

Successfully merging a pull request may close this issue.

2 participants