Remove `@osd/ui-framework` package and replace KUI with OUI

opensearch-project / OpenSearch-Dashboards

📊 Open source visualization dashboards for OpenSearch.

https://opensearch.org/docs/latest/dashboards/index/

Apache License 2.0

1.69k stars 891 forks source link

Remove `@osd/ui-framework` package and replace KUI with OUI #1060

Open tmarkley opened 2 years ago

tmarkley commented 2 years ago

Visualizations, Timeline, Discover are just some of the areas of the code base that reference KUI and the other UI Framework components, but this package is related to multiple security vulnerabilities. We need to get rid of it. I will add more details as we dive deeper.

[ ] #3387
[ ] #3386
[x] #3384
[ ] #3385
[ ] #3828
[x] #3829
[ ] #3830
[ ] #3831
[ ] #3832

Related to #695

joshuarrrr commented 2 years ago

@tmarkley Can this be closed thanks to #1105?

tmarkley commented 2 years ago

@tmarkley Can this be closed thanks to #1105?

No, we still have to replace all of the usages of the KUI components with EUI (or the new design system when we fork) components. See https://github.com/opensearch-project/OpenSearch-Dashboards/blob/89d3872f314aedf9c4f281ea73f784a5802e0175/packages/osd-ui-framework/src/components/index.js#L31-L92

tmarkley commented 1 year ago

Related to https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1560. @ananzh are you working on replacing usages of KUI with OUI as you're removing Angular?

joshuarrrr commented 1 year ago

@tmarkley I made some related subtask issues to better track. And yes, we'll try to opportunistically clean these up when we migrate, deprecate, or rewrite the affected plugins.