owncloud / file-picker

Integrate ownCloud into your own product
https://owncloud.github.io/integration/file_picker/
Apache License 2.0
5 stars 7 forks source link

FP-1.08: Sharing Link Option #246

Open dj4oC opened 1 year ago

dj4oC commented 1 year ago

As a user of the Messenger with the integrated ownCloud Infinite Scale File Picker,
I want the choice to send a sharing link instead of the file itself, with full awareness of the file's current permissions,
So that I can quickly share content while understanding the access constraints for my recipients.


Acceptance Criteria:

  1. Sharing Link Option:

    • When selecting a file / multiple files or a folder / multiple folders to share from the File Picker, I should see two options: "Send File" and "Send Sharing Link."
    • The Messenger should have the option to show this option before opening the file picker to the user and send the result to the file picker
    • The customer should have the option to enable, enforce or disable the sharing link option.
  2. Attention Message:

    • If I opt for "Send Sharing Link," before the link is generated and sent, an attention message should appear, highlighting the existing permissions of the shared content.
    • This attention message should clearly list out the given permissions for the shared file or folder, e.g. "Name", "View", "Edit"
  3. Intuitive Messaging:

    • The attention message should be worded in a customer specific manner, ensuring users, even those unfamiliar with permission settings, can grasp the implications. Example: "Note: You're sharing a link to this file. Some recipients might not have access due to current permissions."

cc @hodyroff @tbsbdr @JammingBen

dj4oC commented 1 year ago

@AversaFTW Please review

AversaFTW commented 1 year ago

What do you mean with "The Messenger should have the option to show this option before opening the file picker to the user and send the result to the file picker"?

dj4oC commented 1 year ago

I thought it would helpful if you could send a parameter to the file picker: "return file" or "return sharing link" In this case you can still use your buttons after starting the process and the UX is smoother

AversaFTW commented 1 year ago

Hm, maybe we can leave this open as a possible addition if the UX calls for it? Because setting the option before opening the filepicker would make having the options inside the filepicker completely redundant. I'd rather not overcomplicate this.

But if it is easier for you to plan, you can already implement this from filepicker side and we can check if we can cleanly display this inside the messenger without having too many buttons.

dj4oC commented 6 months ago

@tbsbdr @kulmann Please check the test result:

Button umbenennen in "Link(s) teilen";
Öffnen eines öffentlichen Freigabelinks: auch bei mehreren Bearbeitern, nur die Eingabe eines einzigen  Namens möglich ("Geben Sie den Namen für Zusammenarbeit ein"); 
doc, ppt, xls können trotz Freigabe zur Bearbeitung nicht bearbeitet werden; 
Freigabe für Eingeladene: Text ändern -> "Berechtigte Personen - Link funktioniert nur für Personen, mit denen die Datei(en) geteilt wurde(n). Anmeldung ist erforderlich.";  
Falls Datei nicht mit der Person, die den Link für berechtigte Personen erhält, geteilt ist -> weißer Bildschirm, Fehlermeldung/Hinweistext sollte aber erscheinen: "Link funktioniert nur für Personen, mit denen die Datei(en) geteilt wurde(n).";
Teilen eines Ordners dauert z.T. sehr lange, ohne dass man weiß, ob etwas passiert (kein Rädchen etc.); 
Wird "Kann hochladen" gewählt -> Dateien können bearbeitet werden, obwohl dies nicht der Fall sein sollte;
File Drop -> wird eine Datei hochgeladen, die den gleichen Namen hat wie eine bereits vorhandene Datei, wird die vorhandene überschrieben, ohne dass eine vorherige Abfrage erfolgt (Ersetzen, Abbrechen, Beide behalten)
kulmann commented 6 months ago

Answering the issues one by one below:

Button umbenennen in "Link(s) teilen"

Makes sense, will do, but this won't be part of ocis v5.0.0. String changes are not allowed after code freeze. Will be there for ocis v5.1.0.

Öffnen eines öffentlichen Freigabelinks: auch bei mehreren Bearbeitern, nur die Eingabe eines einzigen Namens möglich ("Geben Sie den Namen für Zusammenarbeit ein")

There is no such string in the web source code / translations. Is this something from the messenger code base?

doc, ppt, xls können trotz Freigabe zur Bearbeitung nicht bearbeitet werden

Can't reproduce. Would like to kindly ask for a dedicated issue with clear steps to reproduce.

Freigabe für Eingeladene: Text ändern -> "Berechtigte Personen - Link funktioniert nur für Personen, mit denen die Datei(en) geteilt wurde(n). Anmeldung ist erforderlich."

Won't do. Text is fine like it is in the general product.

Falls Datei nicht mit der Person, die den Link für berechtigte Personen erhält, geteilt ist -> weißer Bildschirm, Fehlermeldung/Hinweistext sollte aber erscheinen: "Link funktioniert nur für Personen, mit denen die Datei(en) geteilt wurde(n)."

In ocis v5.0.0 we have a generic error message in place for this case. It's Beim Auflösen des privaten Links ist ein Fehler aufgetreten.. We can't get more specific than this because of technical limitations.

Teilen eines Ordners dauert z.T. sehr lange, ohne dass man weiß, ob etwas passiert (kein Rädchen etc.)

The UI giving more feedback about actions being in progress is tracked here: https://github.com/owncloud/web/issues/9183

Won't be part of oCIS 5.0.0 though.

Wird "Kann hochladen" gewählt -> Dateien können bearbeitet werden, obwohl dies nicht der Fall sein sollte

I think there was an issue that OnlyOffice caches the changes in memory but is not able to write the changes to disk... @micbar do you have an issue link for that? Not an oCIS issue as far as I remember. This is actually a bug in the general product that was not known before. Tracked in https://github.com/owncloud/web/issues/11381

File Drop -> wird eine Datei hochgeladen, die den gleichen Namen hat wie eine bereits vorhandene Datei, wird die vorhandene überschrieben, ohne dass eine vorherige Abfrage erfolgt (Ersetzen, Abbrechen, Beide behalten)

That's again an issue in the messenger implementation. The messenger handles file uploads on its own. Not part of the web embed mode.