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

FP-1.08: Sharing Link Option #246

Open
dj4oC opened this issue Aug 19, 2023 · 6 comments
Open

FP-1.08: Sharing Link Option #246

dj4oC opened this issue Aug 19, 2023 · 6 comments

Comments

@dj4oC
Copy link

dj4oC commented Aug 19, 2023

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
Copy link
Author

dj4oC commented Aug 22, 2023

@AversaFTW
Please review

@AversaFTW
Copy link

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
Copy link
Author

dj4oC commented Aug 24, 2023

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
Copy link

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
Copy link
Author

dj4oC commented Mar 2, 2024

@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
Copy link
Member

kulmann commented Mar 8, 2024

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:
owncloud/web#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 owncloud/web#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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants