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

Allow empty permissions on links again #3079

Merged
merged 3 commits into from
Jul 18, 2022

Conversation

kobergj
Copy link
Contributor

@kobergj kobergj commented Jul 15, 2022

Empty permissions are needed for infamous alias links. Through some PR that functionailty was dropped. Now it is back again

@kobergj kobergj requested review from a team, labkode and ishank011 as code owners July 15, 2022 08:31
@update-docs
Copy link

update-docs bot commented Jul 15, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@kobergj kobergj requested a review from glpatcern as a code owner July 15, 2022 08:33
@pascalwengerter
Copy link
Contributor

Works like a charm 👌🏽

Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: jkoberg <jkoberg@owncloud.com>
@kobergj kobergj force-pushed the AllowEmptyPermissionsForLinksAgain branch from cfa5757 to 3357e6d Compare July 15, 2022 14:03
@kobergj kobergj merged commit 44cb65e into cs3org:edge Jul 18, 2022
@kobergj kobergj deleted the AllowEmptyPermissionsForLinksAgain branch July 18, 2022 07:24
@micbar micbar mentioned this pull request Jul 19, 2022
36 tasks
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

Successfully merging this pull request may close these issues.

3 participants