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
[AppEx-SharedUX] Remove toMountPoint parameter from TableListViewKibanaProvider #182030
[AppEx-SharedUX] Remove toMountPoint parameter from TableListViewKibanaProvider #182030
Conversation
3b592ad
to
0b42eb1
Compare
eff1e14
to
b930894
Compare
/ci |
Pinging @elastic/appex-sharedux (Team:SharedUX) |
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.
Dashboard and maps changes LGTM 👍
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.
Vis changes LGTM 👌🏼
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.
lgtm, sorry for the merge conflict
@@ -26,6 +26,8 @@ | |||
"@kbn/shared-ux-link-redirect-app", | |||
"@kbn/test-jest-helpers", | |||
"@kbn/content-management-table-list-view-common", | |||
"@kbn/core", |
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.
probably nit: I prefer to depend on specific packages types in packages instead of depending on the whole core
…up-table-list-view-kibana-provider-props # Conflicts: # packages/content-management/table_list_view_table/src/services.tsx # packages/content-management/table_list_view_table/tsconfig.json # src/plugins/dashboard/public/dashboard_listing/dashboard_listing_table.tsx
💚 Build Succeeded
Metrics [docs]Module Count
Public APIs missing comments
Async chunks
Page load bundle
Unknown metric groupsAPI count
References to deprecated APIs
History
To update your PR or re-run it, just comment with: |
…naProvider (elastic#182030) ## Summary The `toMountPoint` module from `@kbn/kibana-react-plugin/public` is deprecated. Many areas of code need to include it and provide it as a required prop of `TableListViewKibanaProvider`. This PR resolves that: it shouldn't need to be a required prop anywhere, since it is a stateless component and can be included as-needed. This PR also updates the inclusion of `toMountPoint` to use the non-deprecated source, which is `@kbn/react-kibana-mount`. As a side effect of that, some consumers need to update the `core` property being passed, to ensure it includes the `analytics`, `i18n` and `theme` services. ### Checklist Delete any items that are not applicable to this PR. - [ ] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
Summary
The
toMountPoint
module from@kbn/kibana-react-plugin/public
is deprecated. Many areas of code need to include it and provide it as a required prop ofTableListViewKibanaProvider
. This PR resolves that: it shouldn't need to be a required prop anywhere, since it is a stateless component and can be included as-needed.This PR also updates the inclusion of
toMountPoint
to use the non-deprecated source, which is@kbn/react-kibana-mount
. As a side effect of that, some consumers need to update thecore
property being passed, to ensure it includes theanalytics
,i18n
andtheme
services.Checklist
Delete any items that are not applicable to this PR.