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

[7.11] [Uptime] Fix/details page tabs (#86296) #87796

Merged

Conversation

dominiqueclarke
Copy link
Contributor

Backports the following commits to 7.11:

* Remove tabs from details page

* update

* fix monitord id

* var name

* add Uptime PageHeader tests to test for the presences of tabs or header

* add Uptime MonitorPageTitle test

* Uptime adjust auto generated monitor id regex

* Uptime add tests for MonitorPageTitle to test behavior for missing monitor names and auto generated monitor ids

* remove history from MonitorPageTitle test

* adjust uptime tabs tests

* adjust MonitorPageTitle tests to mock useSelector

* adjust uptime PageHeader tests

* adjust import order in page_header.test

* add props to Uptime PageHeader to determine render, rather than route context

* alphabetize props in Uptime PageHeader

* remove header from individual pages

* add indepdent page header route that matches all paths

* adjust monitor tests to use mockReduxHooks helper, and add mockReactRouterDomHooks

* update tests

* adjust header spacing

Co-authored-by: Shahzad <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
uptime 567 568 +1

Async chunks

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

id before after diff
triggersActionsUi 1.5MB 1.5MB -26.9KB
uptime 1.0MB 1.0MB +193.0B
total -26.7KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
triggersActionsUi 162.0KB 162.1KB +102.0B
Unknown metric groups

async chunk count

id before after diff
triggersActionsUi 31 32 +1

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

@dominiqueclarke dominiqueclarke merged commit a94d216 into elastic:7.11 Jan 11, 2021
@dominiqueclarke dominiqueclarke deleted the backport/7.11/pr-86296 branch January 11, 2021 13:45
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.

2 participants