elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.71k stars 8.13k forks source link

[Dashboard] Simplify module services #167437

Open ThomThomson opened 11 months ago

ThomThomson commented 11 months ago

Problem The Dashboard plugin currently uses an overcomplicated services layer which requires us to re-state all dependencies, their types and their implementations for stubbed usage, for storybooks (which are mostly unused), and for Kibana. This services layer also causes problems in our adoption of certain Shared UX components, as instead of simply passing in the dependency directly, we need to re-state the dependency fully.

Solution Instead of the Presentation Util module services, we should standardize on the approach used in the Embeddables plugin. See src/plugins/embeddable/public/kibana_services.ts for more information.

elasticmachine commented 11 months ago

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