Skip to content

[8.15] [Response Ops][Cases] Spinner on assigning user to case from case page (#191490)#191725

Merged
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-191490
Aug 29, 2024
Merged

[8.15] [Response Ops][Cases] Spinner on assigning user to case from case page (#191490)#191725
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-191490

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

elastic#191490)

## Summary

Fixes elastic#190303

In 8.15 we disabled a request that showed a spinner while React Query's
`isLoading` was true. `isLoading` remains true as long as no data has
been received. Because the request was disabled, `isLoading` stayed true
the entire time, causing the spinner to be displayed instead of the form
that would have activated the request. The fix uses another React Query
state called `isFetching`, which is only true while the request is
actively being made, so initially, no spinner is shown.

## Release Note
Fix endless loading spinner when wanting to add a assignee to a case
from the cases list

(cherry picked from commit 19f6469)
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cases 498.7KB 498.7KB +1.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jcger

@kibanamachine kibanamachine merged commit 33262e0 into elastic:8.15 Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants