Skip to content

feat(Sharing): Add per-recipient permissions - #63854

Merged
susnux merged 9 commits into
masterfrom
unified-sharing-per-recipient-permissions
Sep 1, 2026
Merged

feat(Sharing): Add per-recipient permissions #63854
susnux merged 9 commits into
masterfrom
unified-sharing-per-recipient-permissions

Conversation

@provokateurin

Copy link
Copy Markdown
Member

No description provided.

…deleting all shares

Signed-off-by: provokateurin <kate@provokateurin.de>
…omatically

Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
…s to helper methods

Signed-off-by: provokateurin <kate@provokateurin.de>
…nType

Signed-off-by: provokateurin <kate@provokateurin.de>
…Permission in favor of updateShareProperty and updateSharePermission

Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin added this to the Nextcloud 36 milestone Aug 31, 2026
@provokateurin
provokateurin requested a review from a team as a code owner August 31, 2026 12:53
@provokateurin
provokateurin requested review from Altahrim and sorbaugh and removed request for a team August 31, 2026 12:53
@provokateurin provokateurin added the 3. to review Waiting for reviews label Aug 31, 2026
@provokateurin

Copy link
Copy Markdown
Member Author

/backport to stable35

Comment thread lib/unstable/Sharing/Share.php
Comment thread lib/private/Sharing/SharingBackend.php
@provokateurin
provokateurin force-pushed the unified-sharing-per-recipient-permissions branch from 47701b0 to 52595d0 Compare September 1, 2026 06:18
Signed-off-by: provokateurin <kate@provokateurin.de>
…ingBackend

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurin force-pushed the unified-sharing-per-recipient-permissions branch from 52595d0 to 4d05a09 Compare September 1, 2026 13:53
@provokateurin

Copy link
Copy Markdown
Member Author

@icewind1991 I did one more small thing which is that the check for effective permissions is only done when you want to grant a permission to a recipient. If you yourself are denied a permission, you can deny it for a child recipient of yours.

@susnux
susnux disabled auto-merge September 1, 2026 15:56
@susnux
susnux merged commit 4da8f5a into master Sep 1, 2026
150 checks passed
@susnux
susnux deleted the unified-sharing-per-recipient-permissions branch September 1, 2026 15:56
@susnux

susnux commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

/backport to stable35

@backportbot

backportbot Bot commented Sep 1, 2026

Copy link
Copy Markdown

The backport to stable35 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable35
git pull origin stable35

# Create the new backport branch
git checkout -b backport/63854/stable35

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 362d7365 3e838a2f 48025f62 d0a62e0b fc064129 5567e142 3e9f34ea 5bb0bc0e 4d05a092

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/63854/stable35

Error: Failed to create pull request: Validation Failed: {"resource":"PullRequest","code":"custom","message":"A pull request already exists for nextcloud:backport/63854/stable35."} - https://docs.github.com/rest/pulls/pulls#create-a-pull-request


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants