Skip to content

feat(sharing): Make share permission in bundled edit configurable#57741

Merged
AndyScherzinger merged 1 commit intomasterfrom
feat/configurable-share-permission-in-bundled
Feb 5, 2026
Merged

feat(sharing): Make share permission in bundled edit configurable#57741
AndyScherzinger merged 1 commit intomasterfrom
feat/configurable-share-permission-in-bundled

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Jan 22, 2026

Add config option shareapi_include_share_in_edit to include reshare permission in "Allow editing" bundle.

@nfebe nfebe requested review from a team as code owners January 22, 2026 17:27
@nfebe nfebe added 3. to review Waiting for reviews feature: sharing labels Jan 22, 2026
@nfebe nfebe requested review from come-nc and sorbaugh and removed request for a team January 22, 2026 17:27
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from 8f0083f to 2c1ee9c Compare January 22, 2026 18:04
@AndyScherzinger
Copy link
Member

/backport to stable33

Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

The setting shoud be in the application files_sharing, not in core.

@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch from 2c1ee9c to 2ace2b6 Compare January 26, 2026 22:06
@nfebe nfebe changed the title feat(sharing): make SHARE permission in bundled edit configurable feat(sharing): Make share permission in bundled edit configurable Jan 26, 2026
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from bd1649c to 274dbb8 Compare January 26, 2026 22:39
@nfebe nfebe requested a review from come-nc January 26, 2026 22:39
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 3 times, most recently from 949deb5 to 262cf8c Compare January 29, 2026 08:45
Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

You pushed 3rdparty by mistake.

Also, is it expected that the new option is false by default? I thought the previous behaviour was to include it?

@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 3 times, most recently from fd9f04b to 617c31f Compare January 29, 2026 10:48
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from 7e9ef21 to 3ed65a8 Compare January 29, 2026 14:19
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

LGTM, just one small change please.

@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from d1e35e9 to 79e1214 Compare January 29, 2026 15:00
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch from 79e1214 to f729f27 Compare January 29, 2026 15:05
@nfebe nfebe requested a review from come-nc January 30, 2026 10:31
@nfebe nfebe force-pushed the feat/configurable-share-permission-in-bundled branch from f729f27 to 631f2ef Compare January 30, 2026 10:31
@marcoambrosini marcoambrosini removed their request for review February 4, 2026 09:32
@AndyScherzinger AndyScherzinger force-pushed the feat/configurable-share-permission-in-bundled branch from 631f2ef to b6d612f Compare February 4, 2026 22:24
@AndyScherzinger AndyScherzinger dismissed come-nc’s stale review February 5, 2026 00:11

Changed implementated,

@CarlSchwan CarlSchwan force-pushed the feat/configurable-share-permission-in-bundled branch from b6d612f to fc7c32b Compare February 5, 2026 00:38
@CarlSchwan
Copy link
Member

/compile amend /

@nextcloud-command nextcloud-command force-pushed the feat/configurable-share-permission-in-bundled branch from fc7c32b to a80ff72 Compare February 5, 2026 00:46
Add config option shareapi_bundle_reshare_with_edit to include reshare
permission in "Allow editing" bundle. Default is true to maintain
backward compatibility.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger AndyScherzinger force-pushed the feat/configurable-share-permission-in-bundled branch from a80ff72 to b7dceb0 Compare February 5, 2026 06:36
@AndyScherzinger AndyScherzinger merged commit 9c842fa into master Feb 5, 2026
193 checks passed
@AndyScherzinger AndyScherzinger deleted the feat/configurable-share-permission-in-bundled branch February 5, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

6 participants