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

[embeddable] remove EmbeddableRenderer and embeddable stories #203007

Merged
merged 17 commits into from
Dec 10, 2024

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Dec 4, 2024

PR starts cleaning up legacy embeddable components by removing EmbeddableRenderer, EmbedddableRoot, and embeddable story books.

@nreese
Copy link
Contributor Author

nreese commented Dec 4, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 4, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 4, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 4, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 5, 2024

@elasticmachine merge upstream

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

@elasticmachine merge upstream

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese
Copy link
Contributor Author

nreese commented Dec 6, 2024

/ci

@nreese nreese marked this pull request as ready for review December 6, 2024 18:43
@nreese nreese requested review from a team as code owners December 6, 2024 18:43
@nreese nreese requested a review from a team as a code owner December 6, 2024 18:43
@nreese nreese added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v9.0.0 backport:version Backport to applied version labels v8.17.0 labels Dec 6, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@nreese nreese added release_note:skip Skip the PR/issue when compiling release notes project:embeddableRebuild v8.18.0 and removed v8.17.0 labels Dec 6, 2024
@nreese
Copy link
Contributor Author

nreese commented Dec 9, 2024

@elasticmachine merge upstream

@botelastic botelastic bot added the Feature:Embedding Embedding content via iFrame label Dec 9, 2024
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 9, 2024

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #76 / serverless search UI connectors create and configure connector edit type
  • [job] [logs] FTR Configs #76 / serverless search UI connectors create and configure connector edit type

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
embeddable 144 142 -2

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
embeddable 462 458 -4

Page load bundle

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

id before after diff
embeddable 73.2KB 70.9KB -2.2KB
Unknown metric groups

API count

id before after diff
embeddable 571 564 -7

ESLint disabled line counts

id before after diff
embeddable 6 3 -3

Total ESLint disabled count

id before after diff
embeddable 8 5 -3

History

@nreese nreese changed the title [embeddable] remove EmbeddableRenderer and embeddable stores [embeddable] remove EmbeddableRenderer and embeddable stories Dec 9, 2024
@nreese
Copy link
Contributor Author

nreese commented Dec 10, 2024

@elasticmachine merge upstream

Copy link
Contributor

@Dosant Dosant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Co-authored-by: Anton Dosov <dosantappdev@gmail.com>
@nreese nreese merged commit 80f915f into elastic:main Dec 10, 2024
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12261633628

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 10, 2024
…c#203007)

PR starts cleaning up legacy embeddable components by removing
EmbeddableRenderer, EmbedddableRoot, and embeddable story books.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Anton Dosov <dosantappdev@gmail.com>
(cherry picked from commit 80f915f)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 10, 2024
…203007) (#203655)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[embeddable] remove EmbeddableRenderer and embeddable stories
(#203007)](#203007)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"reese.nathan@elastic.co"},"sourceCommit":{"committedDate":"2024-12-10T17:38:28Z","message":"[embeddable]
remove EmbeddableRenderer and embeddable stories (#203007)\n\nPR starts
cleaning up legacy embeddable components by
removing\r\nEmbeddableRenderer, EmbedddableRoot, and embeddable story
books.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by: Anton Dosov
<dosantappdev@gmail.com>","sha":"80f915f9e3454572d4992591f875b723b8a5ca7c","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Embedding","Team:Presentation","release_note:skip","v9.0.0","project:embeddableRebuild","backport:version","v8.18.0"],"title":"[embeddable]
remove EmbeddableRenderer and embeddable
stories","number":203007,"url":"https://github.com/elastic/kibana/pull/203007","mergeCommit":{"message":"[embeddable]
remove EmbeddableRenderer and embeddable stories (#203007)\n\nPR starts
cleaning up legacy embeddable components by
removing\r\nEmbeddableRenderer, EmbedddableRoot, and embeddable story
books.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by: Anton Dosov
<dosantappdev@gmail.com>","sha":"80f915f9e3454572d4992591f875b723b8a5ca7c"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203007","number":203007,"mergeCommit":{"message":"[embeddable]
remove EmbeddableRenderer and embeddable stories (#203007)\n\nPR starts
cleaning up legacy embeddable components by
removing\r\nEmbeddableRenderer, EmbedddableRoot, and embeddable story
books.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by: Anton Dosov
<dosantappdev@gmail.com>","sha":"80f915f9e3454572d4992591f875b723b8a5ca7c"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <reese.nathan@elastic.co>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…c#203007)

PR starts cleaning up legacy embeddable components by removing
EmbeddableRenderer, EmbedddableRoot, and embeddable story books.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Anton Dosov <dosantappdev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels Feature:Embedding Embedding content via iFrame project:embeddableRebuild release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants