Skip to content

[stable34] fix(files_sharing): Use share id in file actions#62403

Open
backportbot[bot] wants to merge 2 commits into
stable34from
backport/61099/stable34
Open

[stable34] fix(files_sharing): Use share id in file actions#62403
backportbot[bot] wants to merge 2 commits into
stable34from
backport/61099/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 22, 2026

Copy link
Copy Markdown

Backport of #61099

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Remove all the empty commits

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

@backportbot backportbot Bot added bug 3. to review Waiting for reviews labels Jul 22, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.3 milestone Jul 22, 2026
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurin force-pushed the backport/61099/stable34 branch from f02ec78 to b7e6d15 Compare July 22, 2026 11:24
@provokateurin
provokateurin marked this pull request as ready for review July 22, 2026 11:24
@provokateurin
provokateurin requested a review from a team as a code owner July 22, 2026 11:24
@provokateurin
provokateurin requested review from kristian-zendato, nfebe and sorbaugh and removed request for a team July 22, 2026 11:24
@provokateurin
provokateurin enabled auto-merge July 22, 2026 11:24
const node = nodes[0]
const url = generateOcsUrl('apps/files_sharing/api/v1/deletedshares/{id}', {
id: node.id,
id: node.attributes['share-id'],

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least on API level it seems to expect provider:id and id needs to be the group share ID and not the user-group share ID, e.g.

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 bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants