Skip to content

[Cases] Fix severity loading spinner visible while updating tags or assignees bug#158838

Merged
js-jankisalvi merged 2 commits intoelastic:mainfrom
js-jankisalvi:loading-spinner-severity
Jun 2, 2023
Merged

[Cases] Fix severity loading spinner visible while updating tags or assignees bug#158838
js-jankisalvi merged 2 commits intoelastic:mainfrom
js-jankisalvi:loading-spinner-severity

Conversation

@js-jankisalvi
Copy link
Copy Markdown
Contributor

Summary

This PR fixes #157364

loading_spinner_fix.mov

@js-jankisalvi js-jankisalvi added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// Feature:Cases Cases feature labels Jun 1, 2023
@js-jankisalvi js-jankisalvi requested a review from a team as a code owner June 1, 2023 15:13
@js-jankisalvi js-jankisalvi self-assigned this Jun 1, 2023
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

@js-jankisalvi js-jankisalvi changed the title [Cases] Severity loading spinner visible while updating tags or assignees [Cases] Fix severity loading spinner visible while updating tags or assignees bug Jun 1, 2023
@js-jankisalvi js-jankisalvi added the bug Fixes for quality problems that affect the customer experience label Jun 1, 2023
Copy link
Copy Markdown
Member

@cnasikas cnasikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Could you please add a test for the bug fix?

@js-jankisalvi js-jankisalvi enabled auto-merge (squash) June 2, 2023 07:58
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

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 389.7KB 389.8KB +16.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 415 419 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 499 503 +4
total +6

History

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

cc @js-jankisalvi

@js-jankisalvi js-jankisalvi merged commit e3da05c into elastic:main Jun 2, 2023
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Jun 2, 2023
@js-jankisalvi js-jankisalvi deleted the loading-spinner-severity branch October 6, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting bug Fixes for quality problems that affect the customer experience Feature:Cases Cases feature release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Cases] Severity loading spinner visible while updating tag or assignee

5 participants