-
Notifications
You must be signed in to change notification settings - Fork 87
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
Unexpected behavior when creating a group folder with the same name as an existing user folder. #18
Comments
How did you do step 2? Or is this a feature request, that an admin can turn a folder into a group folder? |
There's a weird overlap with existing folders. Take this scenario:
I think Group Folders should act like shares - if somebody shares a folder with the same name as an existing one with you, it should show up besides the existing one, not overwrite it. |
ran into this issue as well, but had an additional side effect of ending up with multiple shared folderswith the same name
the workaround for the duplicated shares is to first save the folder contents, remove the Group Folder, then remove the Shared Folder, then recreate the Group Folder. |
the problem with trying to do any kind of conflict resolution on creating group folders (whether it is moving the files into the group folder, or even just showing a warning), is that it would have to check for conflicts for every user that has the folder which is much to expensive to do on large instances. |
+1 |
This means, that the app has some kind of structural, fundamental issue. IMHO it is not really acceptable to publish this as a "featured" app in the Nextcloud App Store with so many open problems. This app seems to be more like in beta status and not ready for use in production. |
Fixed by #523 |
ref: #647 |
Same issue with v29.0.4.1 : rename an existing "group folder" after the name of an existing fodler shared by individual account. |
Version: latest master...?
The content of the folder should be moved into the "created" groupfolder.
The text was updated successfully, but these errors were encountered: