-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Unified Integrations] Clean up empty states, tutorial links and routing to prefer unified integrations #114911
[Unified Integrations] Clean up empty states, tutorial links and routing to prefer unified integrations #114911
Conversation
…te_links_and_permissions
…te_links_and_permissions
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.
infra
plugin changes LGTM. As noted above, there are some unnecessary page reloads, but they existed before and were not introduced by this PR.
…te_links_and_permissions
…te_links_and_permissions
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / general / Configures a new connector.Cases connectors Configures a new connectorStack Trace
Kibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/ml/jobs/categorization_field_examples·ts.apis Machine Learning jobs Categorization example endpoint - partially valid, more than 75% are nullStandard Out
Stack Trace
Metrics [docs]Module Count
Public APIs missing comments
Any counts in public APIs
Async chunks
Page load bundle
Unknown metric groupsAPI count
async chunk count
History
To update your PR or re-run it, just comment with: cc @snide |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Module Count
Public APIs missing comments
Any counts in public APIs
Async chunks
Page load bundle
Unknown metric groupsAPI count
async chunk count
History
To update your PR or re-run it, just comment with: cc @snide |
…ing to prefer unified integrations (elastic#114911) Cleans up the integrations view and redirects all links to the integration manager.
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
…ing to prefer unified integrations (#114911) (#115493) Cleans up the integrations view and redirects all links to the integration manager. Co-authored-by: Caroline Horn <[email protected]>
Hi @snide Could you please share us info on current user role that restricts him to access integrations and this prompting the required permission error page. Though we attempted to validate above, however could not fetch the required screen page. So, we require more details about user permissions and role that could help us to validate it deeply and with more accuracy. Thanks In advance. QAS |
You'll have to create a user with a role that has "Fleet and integrations" set to "None". |
Hi @cchaos Thanks for the feedback on our query. We are now able to validate the singular graphic/card screen on access to different Kibana modules with limited access.
Further, while performing exploratory we observed issues and thus reported at
Build details: Thanks |
Line item from #114344
Fixes #114079
Fixes #94179
Fixes #94184
Fixes #114924
Adds permissions check directly on the card so that is a use can't access
integrations
they'll see the following.Removes the old beats card in favor of a singular graphic/card. Remove dual title/button CTA in favor of single button
Changes copy / buttons to interstitial
Updates homepage copy and hides it when integrations are unavailable
Removes tabs to tutorial list views, leaving only sample data and upload
Fixes lots of links to now go to integrations rather than the tutorial directory
Removes notices in sample data / upload views
Checklist