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

Upload of folder with same name shows error #6996

Closed
5 tasks
tbsbdr opened this issue May 17, 2022 · 4 comments · Fixed by owncloud/owncloud-design-system#2149 or #7504
Closed
5 tasks

Upload of folder with same name shows error #6996

tbsbdr opened this issue May 17, 2022 · 4 comments · Fixed by owncloud/owncloud-design-system#2149 or #7504
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working

Comments

@tbsbdr
Copy link
Contributor

tbsbdr commented May 17, 2022

Steps to reproduce

  1. Login to https://ocis.ocis-wopi.released.owncloud.works/
  2. Create a new folder "Folder" on the ocis instance and on your local drive.
  3. upload your local "Folder" folder
  4. upload does not start, shows error "Folder "Folder" already exists."
    image

Expected behaviour

System should offer me alternative options to upload the folder.
The appropriate dialogues are consistently shown in case of conflicting resources, eg. in context of:

  • Drag'n drop upload
  • upload via upload-button
  • move-dialog
  • copy-dialog
  • trashbin-restore
  • etc...

General rule

User gets asked what to do in case of conflicting resources, as there is no common pattern across different services/platforms (OneDrive, Google Drive, Box, Windows, Mac, etc. - all different).

Note This dialog concept is also meant to be applied to #1753

Dialogs

image

Behaviour

  1. Replace will replace the existing file (and creates a new version implicitly).
  2. Keep both will create a new resource with an appended number "(xx)" - counting up, if ex. (2) already exists.
  3. Cancel will skip the current resource, if applicable shows the next conflict.
  4. Merge will combine folders with the same name; if they contain conflicting files, user gets asked what to do (dialog for files shows up).
  5. Do this for all XX conflicts will apply the selected option to all XX upcoming conflicts. Is only shown, if there are at least 2 conflicts.
  6. generally applicable: The

Userflows

image


solves:
#3751
#5761
#5106
#6546

groundwork for
#1753

@tbsbdr tbsbdr added Type:Bug Something isn't working Priority:p2-high Escalation, on top of current planning, release blocker labels May 17, 2022
@tbsbdr tbsbdr added this to Qualification in Web Support Board via automation May 17, 2022
@tbsbdr tbsbdr moved this from Qualification to Bugs Prio 2 in Web Support Board May 17, 2022
@lookacat lookacat self-assigned this May 17, 2022
@tbsbdr
Copy link
Contributor Author

tbsbdr commented May 18, 2022

added to initial post:

The appropriate dialogues are consistently shown in case of conflicting resources, eg. in context of:

Drag'n drop upload
upload via upload-button
move-dialog
copy-dialog
trashbin-restore
etc...

@kulmann
Copy link
Member

kulmann commented Sep 5, 2022

We can't implement "merge" with reasonable effort or performance at the moment. Will be ignored for the initial implementation. Instead we're using our usual conflict resolve dialog with Skip (multiple files/folders) / Cancel (single file/folder), Overwrite and Keep both as options.

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Sep 6, 2022

👍 perfect

(I guess you mean Replace - not Overwrite)

@kulmann
Copy link
Member

kulmann commented Sep 6, 2022

(I guess you mean Replace - not Overwrite)

yes 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working
Projects
No open projects
4 participants