Skip to content

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Aug 7, 2025

This commit introduces a change to prevent showing the load more button, if the length of existing results is not equal to the requested limit (which implies it is less than because we never expect it to be more)

Additionally, there is an enhancement to override provider filters passed to the find method. This would improve speed.

@nfebe nfebe requested review from a team, skjnldsv, sorbaugh and susnux and removed request for a team August 7, 2025 06:24
@nfebe nfebe added enhancement 3. to review Waiting for reviews labels Aug 7, 2025
@nfebe nfebe requested a review from a team as a code owner August 7, 2025 06:24
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 📁 Files team Aug 7, 2025
@skjnldsv
Copy link
Member

skjnldsv commented Aug 8, 2025

/compile rebase

@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 Aug 8, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Aug 8, 2025
@skjnldsv skjnldsv enabled auto-merge August 8, 2025 08:06
nfebe and others added 2 commits August 8, 2025 08:07
This commit introduces a change to prevent showing the load more button,
if the length of existing results is not equal to the requested limit (which implies
it is less than because we never expect it to be more)

Additionally, there is an enhancment to override provider filders passed to the find method.
This would improve speed.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command nextcloud-command force-pushed the fix/smarter-loadmore-unified-search branch from 02c659c to 16f80a8 Compare August 8, 2025 08:09
@nextcloud-command nextcloud-command requested a review from a team as a code owner August 8, 2025 08:09
@skjnldsv skjnldsv merged commit 7328754 into master Aug 8, 2025
134 of 136 checks passed
@skjnldsv skjnldsv deleted the fix/smarter-loadmore-unified-search branch August 8, 2025 08:38
@nfebe
Copy link
Contributor Author

nfebe commented Aug 8, 2025

/backport to stable31

@nfebe
Copy link
Contributor Author

nfebe commented Aug 8, 2025

/backport to stable30

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 enhancement feature: search

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

5 participants