-
Notifications
You must be signed in to change notification settings - Fork 16
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
Comments
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. |
Hey team! Please add your planning poker estimate with ZenHub @RyRy79261 @tanmoyAtb @Tbaut |
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
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 |
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 |
@sweetpea22 I'll submit a PR with the following, let's make it nicer then :) link-sharing.mp4 |
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.
The text was updated successfully, but these errors were encountered: