-
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
[embeddable] remove setCustomEmbeddableFactoryProvider from setup API #203853
Conversation
/ci |
/ci |
/ci |
Pinging @elastic/kibana-presentation (Team:Presentation) |
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.
Legacy code removal LGTM! Glad to see the drilldown registration method cleaned up.
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Public APIs missing comments
Public APIs missing exports
Page load bundle
Unknown metric groupsAPI count
ESLint disabled line counts
References to deprecated APIs
Total ESLint disabled count
Unreferenced deprecated APIs
History
|
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.
i18n changes lgtm
Starting backport for target branches: 8.x |
…elastic#203853) Part of elastic#167429 Remove `setCustomEmbeddableFactoryProvider` from embeddable setup API. `setCustomEmbeddableFactoryProvider` only used in `embeddable_enhanced` plugin. Replaced with `initializeReactEmbeddableDynamicActions` in react embeddable system. --------- Co-authored-by: kibanamachine <[email protected]> Co-authored-by: Elastic Machine <[email protected]> (cherry picked from commit 7218d01)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…up API (#203853) (#204073) # Backport This will backport the following commits from `main` to `8.x`: - [[embeddable] remove setCustomEmbeddableFactoryProvider from setup API (#203853)](#203853) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Nathan Reese","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-12T16:11:48Z","message":"[embeddable] remove setCustomEmbeddableFactoryProvider from setup API (#203853)\n\nPart of https://github.com/elastic/kibana/issues/167429\r\n\r\nRemove `setCustomEmbeddableFactoryProvider` from embeddable setup API.\r\n`setCustomEmbeddableFactoryProvider` only used in `embeddable_enhanced`\r\nplugin. Replaced with `initializeReactEmbeddableDynamicActions` in react\r\nembeddable system.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>\r\nCo-authored-by: Elastic Machine <[email protected]>","sha":"7218d01aa42eba6df050262a0c246922d2a0df9d","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 setCustomEmbeddableFactoryProvider from setup API","number":203853,"url":"https://github.com/elastic/kibana/pull/203853","mergeCommit":{"message":"[embeddable] remove setCustomEmbeddableFactoryProvider from setup API (#203853)\n\nPart of https://github.com/elastic/kibana/issues/167429\r\n\r\nRemove `setCustomEmbeddableFactoryProvider` from embeddable setup API.\r\n`setCustomEmbeddableFactoryProvider` only used in `embeddable_enhanced`\r\nplugin. Replaced with `initializeReactEmbeddableDynamicActions` in react\r\nembeddable system.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>\r\nCo-authored-by: Elastic Machine <[email protected]>","sha":"7218d01aa42eba6df050262a0c246922d2a0df9d"}},"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/203853","number":203853,"mergeCommit":{"message":"[embeddable] remove setCustomEmbeddableFactoryProvider from setup API (#203853)\n\nPart of https://github.com/elastic/kibana/issues/167429\r\n\r\nRemove `setCustomEmbeddableFactoryProvider` from embeddable setup API.\r\n`setCustomEmbeddableFactoryProvider` only used in `embeddable_enhanced`\r\nplugin. Replaced with `initializeReactEmbeddableDynamicActions` in react\r\nembeddable system.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>\r\nCo-authored-by: Elastic Machine <[email protected]>","sha":"7218d01aa42eba6df050262a0c246922d2a0df9d"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Nathan Reese <[email protected]>
…elastic#203853) Part of elastic#167429 Remove `setCustomEmbeddableFactoryProvider` from embeddable setup API. `setCustomEmbeddableFactoryProvider` only used in `embeddable_enhanced` plugin. Replaced with `initializeReactEmbeddableDynamicActions` in react embeddable system. --------- Co-authored-by: kibanamachine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Part of #167429
Remove
setCustomEmbeddableFactoryProvider
from embeddable setup API.setCustomEmbeddableFactoryProvider
only used inembeddable_enhanced
plugin. Replaced withinitializeReactEmbeddableDynamicActions
in react embeddable system.