Feat: Support child spaces in `tracimcli user delete -w` - Githubissues

tracim / tracim

Threads, files and pages with status and full history. All in the same place.

https://tracim.fr/

Other

213 stars 37 forks source link

Feat: Support child spaces in `tracimcli user delete -w` #3570

Open grignards opened 4 years ago

grignards commented 4 years ago

Feature description and goals

Child spaces (see #3546) do allow a hierarchy of spaces. The tracimcli user delete -w command deletes a user and all their spaces (-w option).

During a database transaction, for each space to be deleted:

[ ] check if it has sub spaces not owned by the deleted user
[ ] If yes, without -f flag, stop the command with a message explaining the situation (eg. "Space xxx is child of space yyy but its owner is uuu, aborting delete. Use -f flag to force space deletion")
[ ] if yes with -f flag or if no, delete the space and all its sub-spaces

Prerequisites

3546.

lebouquetin commented 4 years ago

@grignards do you plan to have a feature that allow to change parent space?

grignards commented 4 years ago

@grignards do you plan to have a feature that allow to change parent space?

In the first phase I didn't plan this, see #3546. This is one of the points we have to discuss before developing.

lebouquetin commented 4 years ago

I think we should not delete recursively workspaces at the moment. The owner may be another user, so the right way to manage this case, for now is probably to refuse (and to force the CLI user to first delete child space)