Skip to content

[stable32] fix(unified-search): catch errors in response - #62615

Merged
leftybournes merged 2 commits into
stable32from
leftybournes/stable32/fix/unifiedsearch-catch-errors
Jul 28, 2026
Merged

[stable32] fix(unified-search): catch errors in response#62615
leftybournes merged 2 commits into
stable32from
leftybournes/stable32/fix/unifiedsearch-catch-errors

Conversation

@leftybournes

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

When an error response is returned by the unified search api, unified search shows an endless "Searching" message. This catches the error and sets the modal into a non-searching state so it no longer endlessly shows the message.

Not a backport
Based on #61437

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@leftybournes leftybournes added this to the Nextcloud 32.0.14 milestone Jul 28, 2026
@leftybournes
leftybournes requested a review from a team as a code owner July 28, 2026 13:51
@leftybournes
leftybournes requested review from nfebe and removed request for a team July 28, 2026 13:51
@leftybournes

Copy link
Copy Markdown
Member Author

/compile rebase

leftybournes and others added 2 commits July 28, 2026 23:01
Signed-off-by: Kent Delante <kent@delante.me>

When an error response is returned by the unified search api, unified
search shows an endless "Searching" message. This catches the error
and sets the modal into a non-searching state so it no longer
endlessly shows the message.
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command
nextcloud-command force-pushed the leftybournes/stable32/fix/unifiedsearch-catch-errors branch from b0cfccd to 31f2cf7 Compare July 28, 2026 23:04
@leftybournes
leftybournes enabled auto-merge July 28, 2026 23:18
@leftybournes
leftybournes merged commit 44e4637 into stable32 Jul 28, 2026
134 of 137 checks passed
@leftybournes
leftybournes deleted the leftybournes/stable32/fix/unifiedsearch-catch-errors branch July 28, 2026 23:42
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.

4 participants