Skip to content

[stable33] fix(unified-search): catch errors in response - #62614

Open
leftybournes wants to merge 2 commits into
stable33from
leftybournes/stable33/fix/unifiedsearch-catch-errors
Open

[stable33] fix(unified-search): catch errors in response#62614
leftybournes wants to merge 2 commits into
stable33from
leftybournes/stable33/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 33.0.8 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 kristian-zendato, nfebe and sorbaugh and removed request for a team July 28, 2026 13:51
@leftybournes

Copy link
Copy Markdown
Member Author

/compile rebase

@leftybournes
leftybournes enabled auto-merge July 28, 2026 23:44
leftybournes and others added 2 commits July 28, 2026 23:49
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/stable33/fix/unifiedsearch-catch-errors branch from b0c1205 to 5cc590d Compare July 28, 2026 23:52
@leftybournes

Copy link
Copy Markdown
Member Author

Failing tests are unrelated

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