Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Sep 26, 2025

Backport of #55311

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

Todo

  • Review and resolve any conflicts
  • 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 this to the Nextcloud 32 milestone Sep 26, 2025
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@susnux
Copy link
Contributor

susnux commented Oct 1, 2025

/compile

@susnux susnux marked this pull request as ready for review October 1, 2025 00:03
@susnux susnux requested review from a team as code owners October 1, 2025 00:03
@susnux susnux requested review from nfebe and sorbaugh and removed request for a team October 1, 2025 00:03
@susnux susnux force-pushed the backport/55311/stable32 branch from a3a5f12 to 54e694d Compare October 1, 2025 00:08
Add missing `matchUserId` method for full-match-autocomplete options
and update docs to make it clearer what each options exactly means.

Signed-off-by: Ferdinand Thiessen <[email protected]>
When searching we need to:
1. check if sharing is limited to groups
  - if yes only include those
  - otherwise continue
2. check if there are restrictions to groups or phonebook
3. check if full match is included

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the backport/55311/stable32 branch from 54e694d to 919701c Compare October 13, 2025 11:41
@susnux
Copy link
Contributor

susnux commented Oct 13, 2025

/compile

Signed-off-by: nextcloud-command <[email protected]>
@susnux susnux merged commit 3b6bcc7 into stable32 Oct 13, 2025
227 of 233 checks passed
@susnux susnux deleted the backport/55311/stable32 branch October 13, 2025 13:57
@blizzz blizzz mentioned this pull request Oct 14, 2025
Sign up for free to 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.

5 participants