Skip to content

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

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

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

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

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 ownersJanuary 22, 2026 17:27
@nfebe
nfebe requested review from come-nc and sorbaugh and removed request for a teamJanuary 22, 2026 17:27
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from 8f0083f to 2c1ee9cCompareJanuary 22, 2026 18:04
@AndyScherzinger

Copy link
Copy Markdown
Member

/backport to stable33

@come-nccome-nc left a comment

Copy link
Copy Markdown
Contributor

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
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch from 2c1ee9c to 2ace2b6CompareJanuary 26, 2026 22:06
@nfebenfebe changed the title feat(sharing): make SHARE permission in bundled edit configurablefeat(sharing): Make share permission in bundled edit configurableJan 26, 2026
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from bd1649c to 274dbb8CompareJanuary 26, 2026 22:39
@nfebe
nfebe requested a review from come-ncJanuary 26, 2026 22:39
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 3 times, most recently from 949deb5 to 262cf8cCompareJanuary 29, 2026 08:45

@come-nccome-nc left a comment

Copy link
Copy Markdown
Contributor

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?

Comment threadapps/files_sharing/lib/Config/ConfigLexicon.php Outdated
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 3 times, most recently from fd9f04b to 617c31fCompareJanuary 29, 2026 10:48
Comment threadapps/files_sharing/lib/Config/ConfigLexicon.php Outdated
Comment threadapps/files_sharing/src/views/SharingDetailsTab.vue Outdated
Comment threadapps/files_sharing/src/lib/SharePermissionsToolBox.js Outdated
come-nc
come-nc previously requested changes Jan 29, 2026
Comment threadapps/files_sharing/src/lib/SharePermissionsToolBox.js Outdated
Comment threadapps/files_sharing/src/services/ConfigService.ts Outdated
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from 7e9ef21 to 3ed65a8CompareJanuary 29, 2026 14:19

@provokateurinprovokateurin left a comment

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.

LGTM, just one small change please.

Comment threadapps/files_sharing/src/views/SharingDetailsTab.vue Outdated
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch 2 times, most recently from d1e35e9 to 79e1214CompareJanuary 29, 2026 15:00
Comment threadapps/files_sharing/src/lib/SharePermissionsToolBox.js
Comment threadapps/files_sharing/src/views/SharingDetailsTab.vue Outdated
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch from 79e1214 to f729f27CompareJanuary 29, 2026 15:05
@nfebe
nfebe requested a review from come-ncJanuary 30, 2026 10:31
@nfebe
nfebeforce-pushed the feat/configurable-share-permission-in-bundled branch from f729f27 to 631f2efCompareJanuary 30, 2026 10:31
@marcoambrosini
marcoambrosini removed their request for review February 4, 2026 09:32
@AndyScherzinger
AndyScherzingerforce-pushed the feat/configurable-share-permission-in-bundled branch from 631f2ef to b6d612fCompareFebruary 4, 2026 22:24
@AndyScherzinger
AndyScherzinger dismissed come-nc’s stale reviewFebruary 5, 2026 00:11

Changed implementated,

@CarlSchwan
CarlSchwanforce-pushed the feat/configurable-share-permission-in-bundled branch from b6d612f to fc7c32bCompareFebruary 5, 2026 00:38
@CarlSchwan

Copy link
Copy Markdown
Member

/compile amend /

@nextcloud-command
nextcloud-commandforce-pushed the feat/configurable-share-permission-in-bundled branch from fc7c32b to a80ff72CompareFebruary 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
AndyScherzingerforce-pushed the feat/configurable-share-permission-in-bundled branch from a80ff72 to b7dceb0CompareFebruary 5, 2026 06:36
@AndyScherzinger
AndyScherzinger merged commit 9c842fa into masterFeb 5, 2026
193 checks passed
@AndyScherzinger
AndyScherzinger deleted the feat/configurable-share-permission-in-bundled branch February 5, 2026 09:09
@nextcloud-botnextcloud-bot mentioned this pull request Apr 27, 2026
Sign up for freeto 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

@nfebe@AndyScherzinger@CarlSchwan@susnux@provokateurin@come-nc