-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Canvas] Move Handlebars and Flot dependencies out of main bundle #78542
Conversation
@elasticmachine merge upstream |
Pinging @elastic/kibana-canvas (Team:Canvas) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me. Two questions though:
- Will the modules get imported each time the rendering happens? Or is the system smart enough to load just once?
- Will this affect the modules that get bundled in the shareable runtime at all? (i'm not sure how the runtime packager figures out the packages it needs)
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresChrome UI Functional Tests.test/functional/apps/context/_context_navigation·js.context app discover - context - back navigation "before all" hook for "should go back after loading"Standard Out
Stack Trace
Metrics [docs]async chunks size
distributable file count
page load bundle size
History
To update your PR or re-run it, just comment with: |
@poffdeluxe Good Questions.
|
…astic#78542) * Move Handlebars and Flot dependencies out of main bundle * Fix unit test Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
…8542) (#79115) * Move Handlebars and Flot dependencies out of main bundle * Fix unit test Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Moving some dependencies around.