Skip to content

[Observability] [Cases] Fix Cases navigation#102429

Merged
stephmilovic merged 5 commits intoelastic:masterfrom
stephmilovic:obs_cases_onclick_fixx
Jun 21, 2021
Merged

[Observability] [Cases] Fix Cases navigation#102429
stephmilovic merged 5 commits intoelastic:masterfrom
stephmilovic:obs_cases_onclick_fixx

Conversation

@stephmilovic
Copy link
Contributor

Summary

Change from navigateToApp to navigateToUrl to fix issue with breadcrumbs.

Before:
bcbug

After:
navigate_to_url

Checklist

@stephmilovic stephmilovic added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Observability Team label for Observability Team (for things that are handled across all of observability) Feature:Cases Cases feature v7.14.0 Feature:RAC label obsolete labels Jun 16, 2021
@stephmilovic stephmilovic requested review from XavierM and smith June 16, 2021 21:58
@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 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
observability 464.2KB 464.5KB +274.0B

History

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

@stephmilovic stephmilovic merged commit 491b0d1 into elastic:master Jun 21, 2021
@stephmilovic stephmilovic deleted the obs_cases_onclick_fixx branch June 21, 2021 16:54
stephmilovic added a commit to stephmilovic/kibana that referenced this pull request Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Cases Cases feature Feature:RAC label obsolete release_note:skip Skip the PR/issue when compiling release notes Team:Observability Team label for Observability Team (for things that are handled across all of observability) v7.14.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants