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

Embeddables 👉 New Platform #46771

Merged
merged 14 commits into from
Sep 27, 2019
Merged

Embeddables 👉 New Platform #46771

merged 14 commits into from
Sep 27, 2019

Conversation

streamich
Copy link
Contributor

@streamich streamich commented Sep 27, 2019

Summary

  • Moves Embeddables NP-ready plugin to NP.
  • No code/business logic changes here.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers


Dev Docs

Embeddables API has been moved from src/legacy/core_plugins/embeddable_api to the embeddable New Platform plugin at src/plugins/embeddable.

Now you can use Embeddables API in the New Platform by depending on embeddable plugin and receiving it through the plugin dependencies. In the legacy Kibana platform you can access Embeddables though ui/new_platform backdoor.

import { npSetup, npStart } from 'ui/new_platform';

npSetup.plugins.embeddable
npStart.plugins.embeddable

@streamich streamich requested a review from a team as a code owner September 27, 2019 10:35
@streamich streamich added Feature:Embedding Embedding content via iFrame Feature:NP Migration release_note:skip Skip the PR/issue when compiling release notes Team:AppArch v7.5.0 labels Sep 27, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@streamich streamich requested review from stacey-gammon, ppisljar and a team September 27, 2019 12:17
Copy link
Contributor

@ryankeairns ryankeairns left a comment

Choose a reason for hiding this comment

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

LGTM, just moved a couple of SASS files.

@@ -2,7 +2,8 @@
"id": "advanced_ui_actions",
"version": "kibana",
"requiredPlugins": [
"embeddable"
"embeddable",
"ui_actions"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this will need to be uiActions, but until it's used... not really important.

Copy link
Contributor

@stacey-gammon stacey-gammon left a comment

Choose a reason for hiding this comment

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

lgtm, I'm assuming follow up will be to remove the re-exporting from legacy paths, update all import paths, and keep that legacy plugin around only with the css?

btw, nice to see you back in action! Welcome back!

@streamich streamich merged commit f7b95ec into elastic:master Sep 27, 2019
@streamich
Copy link
Contributor Author

@stacey-gammon yes, there will be follow-up PRs.

streamich added a commit that referenced this pull request Sep 27, 2019
* chore: 🤖 move embeddable_api to NP "embeddable" plugin

* fix: 🐛 fix TypeScript type check errors

* fix: 🐛 fix TypeScript and build errors

* test: 💍 fix Jest tests

* fix: 🐛 fix translations

* test: 💍 add Mocha mocks

* fix: 🐛 change import syntax from function test sample plugin

* fix: 🐛 revert canvas changes back and leave placeholders

* chore: 🤖 revert more changes

* chore: 🤖 revert more changes

* fix: 🐛 fix TypeScript type check errors

* fix: 🐛 fix functional test imports

* fix: 🐛 fix x-pack Jest test importa

* test: 💍 fix advanced_ui_actiosn import for tests
@streamich streamich changed the title Embeddables 2 np Embeddables 👉 New Platform Oct 1, 2019
@streamich streamich added release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. and removed release_note:skip Skip the PR/issue when compiling release notes labels Oct 1, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Embedding Embedding content via iFrame Feature:NP Migration release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. v7.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants