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.x] [Core UI] Kibana Overview Page (#75827) #79708

Merged
merged 3 commits into from
Oct 6, 2020

Conversation

cqliu1
Copy link
Contributor

@cqliu1 cqliu1 commented Oct 6, 2020

Backports the following commits to 7.x:

* Added kibana landing page

Created kivana_overview plugin

Removed test from home plugin

Added CSS

Fixed page header links

Added news feed

Fixed spacers between news items

[Core UI] Kibana Overview Page Style Tweaks (elastic#76712)

Fixed link to index management

Added solution cards to kibana landing page

Added solution links

Fixed ts errors

Using publishReplay() to support multiple consumers in newsfeed plugin

Added createNewsFeed$ to newsfeed plugin start

Added tests

Removed unnecessary export

Hides overview link when other Kibana apps are not available

Added icon to overview plugin

Removed question mark from news feed title

Updated plugin-list

Fixed i18n errors

Revert snapshot

Updated getting started page copy

Hide news feed when no news feed results

Disables Kibana overview page when kibana apps are unavailable

Updated snapshots

Refactor to use KibanaContextProvider

Fixed security tests

Fixed newsfeed api test

Moved overview_footer and overview_header to kibana-react plugin

[Core UI] Kibana Overview Page Style Fixes (elastic#78677)

* Fixed a11y issues

* Made newsfeed optional dep of kibana overview plugin

* Removed duplicate license copy

* Fixed management security test

* Added toast to change default route button

* Updated snapshots

* Simplified toast notification

* Fixed i18n error

* Assigned kibana_overview plugin to Core UI in CODEOWNERS

* Updated snapshots

* Fix import

* [Core UI] Kibana Overview Page Style Fixes, Part 3 (elastic#78970)

* fix overview cards not stretching height equally

* change var name for better specificity

* [Core UI] Kibana Overview Page Style Fixes, Part 4 (elastic#79136)

* Adds support for all newsfeed plugin config settings in createNewsFeed$

* Fixed type

* Updated kibana overview page route

* Fixed imports in page_footer and page_header

* Update Kibana overview graphics (elastic#79534)

* Updated snapshots

* Updated snapshots

* Changes newsfeed endpoint to kibana analytics in kibana_overview plugin

* Renamed components

* Fixed overview page footer and header component class names

* Removed extraneous files

* Fixed import

* Replaced SVGs with optimized SVGs

* Fixed header and footer in home and kibana overview pages

* Updated snapshots

* Changed url_forwarding plugin appRoute

* Fixed aria-labelledby value

* Updated snapshots

* Added base paths

Co-authored-by: Michael Marcialis <[email protected]>
Co-authored-by: Ryan Keairns <[email protected]>
# Conflicts:
#	.github/CODEOWNERS
@cqliu1 cqliu1 added the backport label Oct 6, 2020
@cqliu1
Copy link
Contributor Author

cqliu1 commented Oct 6, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
kibanaOverview - 32 +32
kibanaReact 293 307 +14
newsfeed 10 16 +6
total +52

async chunks size

id before after diff
advancedSettings 892.1KB 892.2KB +148.0B
home 413.4KB 404.0KB -9.4KB
kibanaOverview - 28.4KB +28.4KB
kibanaReact 358.7KB 358.7KB +21.0B
ml 10.6MB 10.6MB +172.0B
total +19.3KB

distributable file count

id before after diff
default 48054 48087 +33
oss 29366 29399 +33

page load bundle size

id before after diff
canvas 1.0MB 1.0MB +186.0B
dashboard 350.7KB 350.8KB +184.0B
discover 87.9KB 88.0KB +163.0B
enterpriseSearch 20.0KB 20.3KB +257.0B
graph 15.8KB 16.1KB +289.0B
home 26.7KB 26.0KB -711.0B
kibanaOverview - 40.5KB +40.5KB
kibanaReact 124.4KB 143.9KB +19.5KB
maps 164.6KB 164.8KB +174.0B
newsfeed 22.1KB 26.6KB +4.5KB
observability 52.4KB 52.6KB +243.0B
securitySolution 587.1KB 587.4KB +238.0B
urlForwarding 17.1KB 17.2KB +25.0B
total +65.5KB

History

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

@cqliu1 cqliu1 merged commit 6e16d85 into elastic:7.x Oct 6, 2020
@cqliu1 cqliu1 deleted the backport/7.x/pr-75827 branch October 6, 2020 20:20
@cqliu1 cqliu1 restored the backport/7.x/pr-75827 branch June 8, 2022 13:59
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