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

[share-dialog-redesign] IV: Permission "Internal" #8752

Closed
hurradieweltgehtunter opened this issue Mar 30, 2023 · 3 comments
Closed

[share-dialog-redesign] IV: Permission "Internal" #8752

hurradieweltgehtunter opened this issue Mar 30, 2023 · 3 comments

Comments

@hurradieweltgehtunter
Copy link
Contributor

hurradieweltgehtunter commented Mar 30, 2023

This is a sub-task of #8743

While screening all available permission when sharing resources, I stumbled upon the permission “Internal” (Sharing via link).

I cannot get my head wrapped this “permission”. As far as I understand, this is a link to a resource within ownCloud which requires the user to be logged in, like the private link in oc10?
It seems to me that this is misplaced in the “permissions” section when sharing because technically it's not a permission but a requirement. While the other permissions, for example “Editor”, grants the user the right to edit a resource, the “permission” “internal” does not grant anything but requires the user to be logged in. Therefore, it says nothing about what the user is allowed to do → Not a permission.

Please correct me, if I misunderstood this.
But If I'm not mistaken, the “Internal” should not be listed in the permissions.

Proposals:

  • Add it to sharing links as optional option “[ ] Requires login” (not sure if it makes sense to share a resource via link with internal users... ? )
  • If it is the private link from oc10 add it—parallel to the quick link—as separate link type.
@kulmann
Copy link
Member

kulmann commented Mar 30, 2023

Your understanding of the Internal link role is correct. We already had the realization that it is misplaced in the links section. It only serves as a kind of "pointer" for the people that have access via a share or a space membership.

The underlying idea was, that we don't want to confront the user with multiple types of links (private vs public link) anymore, because that's also hard to grasp as a concept. oCIS doesn't show multiple link types anymore, only a link, which has the /s/<linkToken> URL syntax, independent from the role. But it turned out that the internal role is hard to grasp in a similar way to the private link.

@hurradieweltgehtunter
Copy link
Contributor Author

hurradieweltgehtunter commented Mar 30, 2023

Thanks for the info!
Please have a look at: #8750 -> Quicklink

Can't this be combined? Quicklink is the internal pointer, not created on data layer but from string concat. end of story.
If one needs sharing links with permissions, create one.

This would condense the (not easy to explain) quicklink and the (not really a) permission internal to one, more simple concept.

@kulmann
Copy link
Member

kulmann commented Jun 4, 2024

Share dialog redesign is not on the roadmap. What we currently have works for now. Thanks for the input, maybe later when we come back to this in the future.

@kulmann kulmann closed this as not planned Won't fix, can't repro, duplicate, stale Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants