Skip to content

fix: do not show account filter for public shares - #52533

Merged
skjnldsv merged 2 commits into
masterfrom
fix/no-account-filter-public-share
May 15, 2025
Merged

fix: do not show account filter for public shares#52533
skjnldsv merged 2 commits into
masterfrom
fix/no-account-filter-public-share

Conversation

@susnux

@susnuxsusnux commented Apr 29, 2025

Copy link
Copy Markdown
Contributor

Summary

This makes no sense for public shares as there are no known users to the share receiver.

Checklist

@susnuxsusnux added this to the Nextcloud 32 milestone Apr 29, 2025
@susnux
susnux requested a review from skjnldsvApril 29, 2025 14:29
@susnux
susnux requested a review from a team as a code ownerApril 29, 2025 14:29
@susnux
susnux requested review from artonge and sorbaugh and removed request for a teamApril 29, 2025 14:29
@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable31

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the fix/no-account-filter-public-share branch from 8076854 to e9382dcCompareMay 13, 2025 12:21
@susnux

Copy link
Copy Markdown
ContributorAuthor

/compile

@susnux
susnux requested review from provokateurin and removed request for sorbaughMay 13, 2025 12:22
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv
skjnldsv merged commit 183c253 into masterMay 15, 2025
@skjnldsv
skjnldsv deleted the fix/no-account-filter-public-share branch May 15, 2025 17:25
@skjnldsvskjnldsv mentioned this pull request Aug 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@susnux@artonge@nfebe@skjnldsv@provokateurin@nextcloud-command