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

Display current links for a shared folder and allow to revoke them #1593

Closed
Tbaut opened this issue Sep 28, 2021 · 8 comments
Closed

Display current links for a shared folder and allow to revoke them #1593

Tbaut opened this issue Sep 28, 2021 · 8 comments
Assignees
Labels
Epic: Sharing with link Added to issues due for Sharing v1, allowing to share with any email from Google/Github. Product: Files

Comments

@Tbaut
Copy link
Collaborator

Tbaut commented Sep 28, 2021

For the owner of a shared folder, it is possible to create a link sharing for this bucket.
The "Manage bucket" modal should show any link sharing currently active, and allow the owner to delete them.

needs clarification: We should actually also let an owner create a link after the folder creation, most probably at the same place. We could have a dedicated issue for this.

Note - This issue does not include the changes to how existing share users are displayed, only the display/copying of a link.

@Tbaut Tbaut added Planning: Needs Design 🖼️ This UI component still needs one more designs to complete Planning: Needs Clarification 🔍 Added to issues relating to a low severity bugs. Epic: Sharing with link Added to issues due for Sharing v1, allowing to share with any email from Google/Github. Product: Files labels Sep 28, 2021
@Tbaut Tbaut changed the title Diplay current links for a shared folder and allow to revoke them Display current links for a shared folder and allow to revoke them Sep 28, 2021
@sweetpea22
Copy link
Contributor

sweetpea22 commented Oct 6, 2021

User opens manage share and has an active link. If no link is active, link management modal still appears, but "active link" line will change to "copy link" or "generate link".
Screen Shot 2021-10-06 at 9 47 55 AM

If user retroactively adds edit permissions to an active link, we can indicate state change with button text content changing or color change.
Edit link

Manage members can look like this, as the tag input could get unruly when lots of people are involved. Full access gives edit rights, plus share rights.
Permissions toggle
Remove user

open to changes as always

@Tbaut
Copy link
Collaborator Author

Tbaut commented Oct 6, 2021

This is a pretty big change, on top of adding the link sharing feature. I'll tackle the redesign in a separate issue I think.
Also just a nit, but we said the "Full access" can't be given to someone this way, at least not for now.

@Tbaut Tbaut removed Planning: Needs Design 🖼️ This UI component still needs one more designs to complete Planning: Needs Clarification 🔍 Added to issues relating to a low severity bugs. labels Oct 6, 2021
@FSM1
Copy link
Contributor

FSM1 commented Oct 6, 2021

Hey team! Please add your planning poker estimate with ZenHub @RyRy79261 @tanmoyAtb @Tbaut

@Tbaut
Copy link
Collaborator Author

Tbaut commented Oct 6, 2021

Another suggestion @sweetpea22, we can have several links, say one for view only, the other for edit. So the design should maybe cater for several links. Could we have all the important things on 1 line

  • link (truncated) - permission dropdown - Button to copy link - X to revoke ?

Also we don't have any other toggle in the UI, and I'm not sure this is well suited, when for the same permission setting, we have 2 different UI elements (a toggle and a dropdown)

@sweetpea22
Copy link
Contributor

sweetpea22 commented Oct 6, 2021

Also we don't have any other toggle in the UI, and I'm not sure this is well suited, when for the same permission setting, we have 2 different UI elements (a toggle and a dropdown)

Right, so I think toggle only makes sense if we republish permission changes to a single link because then users can be sure that one-click switch applies to the one link. But if we have multiple links it would make sense to use this dropdown, gives a better sense of differentiating what is selected. I'll get back to you with examples like you described

@Tbaut
Copy link
Collaborator Author

Tbaut commented Oct 6, 2021

Nice thanks! Looking at something like this, maybe with a button to create a new link as well
image

@Tbaut
Copy link
Collaborator Author

Tbaut commented Oct 7, 2021

I'll change this slightly, because changing the permission of a link is actually not supported, so I'll prevent this from happening, and just make sure the permission is set at the creation

@Tbaut
Copy link
Collaborator Author

Tbaut commented Oct 7, 2021

@sweetpea22 I'll submit a PR with the following, let's make it nicer then :)

link-sharing.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: Sharing with link Added to issues due for Sharing v1, allowing to share with any email from Google/Github. Product: Files
Projects
None yet
Development

No branches or pull requests

3 participants