-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[new-platform] Decouple ui/chrome from angular #34091
Comments
Pinging @elastic/kibana-platform |
Categories of Angular things in ui/chromeHere I've done a deep dive into all the things that ui/chrome does. I'm sharing this to help clarify what's going on with others. In a follow up, I'll start to put together a plan of how we tackle this. I'll try to keep this up to date as I learn more, but you can also follow along here. Static config
Global angular setup/configuration
UI
BootstrappingEntry point flow
chrome.dangerouslyGetActiveInjector
Shimmed APIsAPIs that are powered by Core, but exposed to the legacy platform through ui/chrome need to be removed by refactoring legacy plugins to import them from This is non-critical for getting rid of Angular, but is needed to get rid of this module completely. |
For many of the static values provided via |
Most of this has been completed and there's a lot of overlap now with this and the plans in #18843. Closing. |
We need to decouple all the functionality that ui/chrome provides via Angular. This includes:
kibana
angular modulekibana
angular moduleIn addition, we need to remove the angular bootstrapping code from chrome and into each application plugin. This will enable applications that do not require angular to completely decouple themselves from it once the new platform provides an Application Service for loading apps (#18843).
This issue will be fleshed out further and broken down as I dive into ui/chrome to identify the major pieces that need to be removed and how that will need to be coordinated across the project.
There are also a few non-angular APIs still provided by the legacy chrome that need to moved over to the new platform (#20696). This may or may not be coupled to this effort.
Execution Plan
The text was updated successfully, but these errors were encountered: