Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable24] Add share search tweaks #32360

Merged
merged 4 commits into from
May 13, 2022

Conversation

artonge
Copy link
Contributor

@artonge artonge commented May 12, 2022

artonge added 3 commits May 12, 2022 15:05
shareapi_restrict_user_enumeration_full_match_ignore_second_display_name was introduced to ignore second display name during search from the share panel. But this setting was not respected by search from the calendar application. This fix it.

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge self-assigned this May 12, 2022
@artonge artonge added this to the Nextcloud 24.0.1 milestone May 12, 2022
@artonge artonge requested review from a team, PVince81, ArtificialOwl and skjnldsv and removed request for a team May 12, 2022 13:08
@artonge artonge changed the base branch from master to stable24 May 12, 2022 13:08
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the backport/stable24/share_search_tweaks branch from 3ad73d1 to 9a131a1 Compare May 12, 2022 13:09
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 13, 2022
@skjnldsv skjnldsv merged commit f02ccc6 into stable24 May 13, 2022
@skjnldsv skjnldsv deleted the backport/stable24/share_search_tweaks branch May 13, 2022 18:15
@blizzz blizzz mentioned this pull request May 16, 2022
* Check if the search should match the email
*
* @return bool
* @since 25.0.0
Copy link
Member

Choose a reason for hiding this comment

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

24.0.1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix here: #32739

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish feature: search feature: sharing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants