Skip to content

fix(ShareApiController): fix listing of remote shares for the owner#59163

Open
szaimen wants to merge 1 commit intomasterfrom
enh/noid/fix-remote-shares-in-controller
Open

fix(ShareApiController): fix listing of remote shares for the owner#59163
szaimen wants to merge 1 commit intomasterfrom
enh/noid/fix-remote-shares-in-controller

Conversation

@szaimen
Copy link
Contributor

@szaimen szaimen commented Mar 23, 2026 •
edited
Loading

Fixes the following problem:

Create a folder (called "Door delen"), which I shared as Projectadmin1 with DashboardAdmin1, with all permissions.

DashboardAdmin1 reshares this folder;

  1. Local with Wes Schibo
  2. Via mail to a user
  3. Via public link
  4. Via OCM FedShare with user@example.com

As Projectadmin1, I can only see reshare number 1, 2 and 3, not the federated reshare (4)

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@szaimen szaimen added this to the Nextcloud 34 milestone Mar 23, 2026
@szaimen
Copy link
Contributor Author

szaimen commented Mar 23, 2026

/backport! to stable32

@backportbot backportbot bot mentioned this pull request Mar 23, 2026
Draft
@szaimen szaimen force-pushed the enh/noid/fix-remote-shares-in-controller branch 2 times, most recently from 1c2b09e to a34738e Compare March 23, 2026 18:48
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 23, 2026
@szaimen szaimen marked this pull request as ready for review March 23, 2026 19:12
@szaimen szaimen requested review from a team, SebastianKrupinski and kesselb as code owners March 23, 2026 19:12
@szaimen szaimen requested review from Altahrim, icewind1991, leftybournes and salmart-dev and removed request for a team March 23, 2026 19:12
@szaimen
fix(ShareApiController): fix listing of remote shares for the owner
57967bb 
Signed-off-by: Simon L. <szaimen@e.mail.de>
@szaimen szaimen force-pushed the enh/noid/fix-remote-shares-in-controller branch from a34738e to 57967bb Compare March 23, 2026 19:38
@szaimen
Copy link
Contributor Author

szaimen commented Mar 23, 2026

/backport to stable33

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

Reviewers

@kesselb kesselb Awaiting requested review from kesselb

@SebastianKrupinski SebastianKrupinski Awaiting requested review from SebastianKrupinski

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend

@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend

@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@provokateurin provokateurin Awaiting requested review from provokateurin

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

3. to review Waiting for reviews AI assisted backport-request bug

Projects

None yet

Milestone

Nextcloud 34

Development

Successfully merging this pull request may close these issues.

1 participant

@szaimen