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

Can't create new workspace on app.twenty-main.com after deleting workspace #9728

Closed
BOHEUS opened this issue Jan 17, 2025 · 1 comment · Fixed by #9741
Closed

Can't create new workspace on app.twenty-main.com after deleting workspace #9728

BOHEUS opened this issue Jan 17, 2025 · 1 comment · Fixed by #9741
Assignees

Comments

@BOHEUS
Copy link
Contributor

BOHEUS commented Jan 17, 2025

Scenario:

  1. Open app.twenty-main.com
  2. Use different email, e.g. [email protected] to create a new workspace
  3. After creating a workspace, delete it
  4. Go again to app.twenty-main.com and try to create a new workspace with the same email used in step 2

Actual: New workspace can't be created with the same email after workspace is deleted

Expected: New workspace can be created with the same email after workspace is deleted

@etiennejouan
Copy link
Contributor

etiennejouan commented Jan 20, 2025

From what I understand, user (who has create a workspace and deleted it) can't create new workspace because after deleting his previous workspace, user has not been soft deleted.

There is several cases where user should be soft deleted :
1/ if user delete his account
2/ if a user's workspace is deleted + user has no other workspace
3/ If a workspaceMember is removed + user has no other workspace

Hi @BOHEUS 👋
I'm digging in the codebase and I've related question : I don't understand how userWorkspace record can be deleted for a user deleting his account (from a workspace with other workspace members) - I don't find any clue in the deleteUser user service method logic.

Edit : I've just noticed the database event listener ! Thank you

@ijreilly ijreilly moved this to 🏗 In Progress in 🎯 Roadmap & Sprints Jan 20, 2025
@ijreilly ijreilly moved this from 🏗 In Progress to 👀 In review in 🎯 Roadmap & Sprints Jan 20, 2025
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in 🎯 Roadmap & Sprints Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants