Skip to content

[stable29] fix(unified_search): Maintain filters when loading more results - #52898

Closed
backportbot[bot] wants to merge 3 commits into
stable29from
backport/52772/stable29
Closed

[stable29] fix(unified_search): Maintain filters when loading more results#52898
backportbot[bot] wants to merge 3 commits into
stable29from
backport/52772/stable29

Conversation

@backportbot

@backportbotbackportbotBot commented May 16, 2025

Copy link
Copy Markdown

Backport of #52772

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.

nfebeand others added 3 commits May 16, 2025 10:28
…results
Maintain existing non-provider filters (like date and person filters) while
loading more results for a specific provider.
This now works as the provider data passed to the `newResults` array after search
is built dynamically and not manually, the result of a manual build was that the
`provider.filters` properpy which is used in the `find` method was being left out
in the providers pushed into the results array.
Fixes#52081
Signed-off-by: nfebe <fenn25.fn@gmail.com>
Before now the result limit is never reset until the page is refreshed.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@backportbot
backportbotBot requested review from nfebe, skjnldsv and susnuxMay 16, 2025 10:28
@backportbotbackportbotBot added the 3. to review Waiting for reviews label May 16, 2025

@nfebenfebe 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.

Needs closer review or backport from 30

@skjnldsv

Copy link
Copy Markdown
Member

Closing as stale and EOL branch.
If still required, please resume your work on it by rebasing and requesting for reviews again 🙏

@susnux
susnux deleted the backport/52772/stable29 branch November 1, 2025 11:28
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@nfebe@nextcloud-command