Skip to content

[7.x] App Services: Remove remaining uiActions, expressions, data, embeddable circular dependencies. (#82791)#86608

Merged
lukeelmers merged 1 commit intoelastic:7.xfrom
lukeelmers:backport/7.x/pr-82791
Dec 20, 2020
Merged

[7.x] App Services: Remove remaining uiActions, expressions, data, embeddable circular dependencies. (#82791)#86608
lukeelmers merged 1 commit intoelastic:7.xfrom
lukeelmers:backport/7.x/pr-82791

Conversation

@lukeelmers
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…le circular dependencies. (elastic#82791)

* Move applyFilter, selectRange, valueClick triggers to data/embeddables.

* Update imports.

* Remove embeddable references to non-existent data plugin dependency.

* remove data mocks from embeddable

* Remove query, filters, timeRange from EmbeddableInput and move to apps.

* Remove data plugin imports from embeddable test samples.

* Remove circular dependencies caused by expressions renderer handlers.

* Update circular deps allowList.

* Remove data dependency on embeddable.

* Revert accidental data plugin change.

* Fix new circular deps issues.

* Update generated docs.

* Fix type errors in vis_type_xy

* Fix inspector data table.
@lukeelmers lukeelmers added the backport This PR is a backport of another PR label Dec 20, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
data 610 612 +2
uiActions 30 27 -3
total -1

Async chunks

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

id before after diff
discover 453.6KB 453.5KB -51.0B
maps 2.8MB 2.8MB -80.0B
triggersActionsUi 1.6MB 1.5MB -26.9KB
visTypeVislib 643.8KB 643.7KB -49.0B
visualizations 61.7KB 61.7KB +1.0B
total -27.1KB

Distributable file count

id before after diff
default 47483 48246 +763

Page load bundle

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

id before after diff
dashboardEnhanced 34.8KB 34.5KB -284.0B
data 1005.3KB 1005.7KB +465.0B
discoverEnhanced 16.0KB 15.7KB -281.0B
embeddable 191.7KB 196.6KB +4.9KB
triggersActionsUi 162.6KB 162.7KB +102.0B
uiActions 69.4KB 63.6KB -5.7KB
urlDrilldown 51.2KB 51.2KB +26.0B
visualizations 169.3KB 169.3KB -25.0B
total -822.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

@lukeelmers lukeelmers merged commit 620f532 into elastic:7.x Dec 20, 2020
@lukeelmers lukeelmers deleted the backport/7.x/pr-82791 branch December 20, 2020 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants