Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Security Solutions] Fix Serverless page title #163911

Merged
merged 2 commits into from
Aug 16, 2023
Merged

Conversation

machadoum
Copy link
Member

@machadoum machadoum commented Aug 15, 2023

issue: #163664

Summary

Refactor useUpdateBrowserTitle to use pathname instead of SpyRoute

Checklist

Delete any items that are not applicable to this PR.

@machadoum machadoum force-pushed the 157847_fix_page_title branch from c44705c to a34e9e6 Compare August 15, 2023 07:49
@machadoum machadoum self-assigned this Aug 15, 2023
@machadoum machadoum added Team:Threat Hunting Security Solution Threat Hunting Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore Project:Serverless Work as part of the Serverless project for its initial release bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes labels Aug 15, 2023
@machadoum machadoum marked this pull request as ready for review August 15, 2023 07:51
@machadoum machadoum requested a review from a team as a code owner August 15, 2023 07:51
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 4416 4417 +1

Async chunks

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

id before after diff
securitySolution 15.6MB 15.6MB +423.0B

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

cc @machadoum

@angorayc
Copy link
Contributor

Tested locally, LGTM! Thanks for fixing this Pablo 👍

@machadoum machadoum merged commit a41efea into main Aug 16, 2023
@machadoum machadoum deleted the 157847_fix_page_title branch August 16, 2023 06:18
@kibanamachine kibanamachine added v8.10.0 backport:skip This commit does not require backporting labels Aug 16, 2023
hop-dev pushed a commit to hop-dev/kibana that referenced this pull request Aug 16, 2023
## Summary

Refactor useUpdateBrowserTitle to use pathname instead of SpyRoute



### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
bryce-b pushed a commit that referenced this pull request Aug 22, 2023
## Summary

Refactor useUpdateBrowserTitle to use pathname instead of SpyRoute



### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting bug Fixes for quality problems that affect the customer experience Project:Serverless Work as part of the Serverless project for its initial release release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore Team:Threat Hunting Security Solution Threat Hunting Team v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants