Skip to content

[8.x] [Data View Mgmt] Implement state service (#193660)#194022

Merged
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-193660
Sep 30, 2024
Merged

[8.x] [Data View Mgmt] Implement state service (#193660)#194022
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-193660

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

Intermediate step in converting Data View Management to DataViewLazy.
Moved `useCallback` functionality into central service so much of the
state functionality is in one place and easier to understand and
refactor.

Part of elastic#178926

Broken out from elastic#190292 which
tried to do everything in one go and became too messy.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Julia Rechkunova <julia.rechkunova@gmail.com>
(cherry picked from commit d772a11)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 25, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) September 25, 2024 14:42
@mattkime
Copy link
Copy Markdown
Contributor

/ci

@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
dataViewManagement 236 239 +3

Async chunks

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

id before after diff
dataViewManagement 140.2KB 142.4KB +2.2KB

Page load bundle

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

id before after diff
dataViewManagement 5.0KB 5.2KB +139.0B
dataViews 61.5KB 61.6KB +108.0B
total +247.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
dataViewManagement 3 2 -1

References to deprecated APIs

id before after diff
dataViewManagement 45 35 -10

Total ESLint disabled count

id before after diff
dataViewManagement 3 2 -1

History

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

cc @mattkime

@kibanamachine kibanamachine merged commit 0308537 into elastic:8.x Sep 30, 2024
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.

3 participants