-
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
[Visualize] Renders no data component if there is no ES data or dataview #132223
Conversation
@@ -125,6 +126,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) { | |||
}); | |||
|
|||
it('Does not show dashboard save modal when on quick save', async function () { | |||
await esArchiver.loadIfNeeded('test/functional/fixtures/es_archiver/logstash_functional'); |
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.
ℹ️ I found out that on this test, there were no es data loaded, and with the change done on this PR, we need es data in order to have access to the viz editor. So I am loading them on this test.
await security.testUser.setRoles(['global_maps_all', 'global_visualize_all'], { | ||
skipBrowserRefresh: true, | ||
}); | ||
await security.testUser.setRoles( |
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.
ℹ️ I am adding another role to this user here. The test_logstash_reader
adds permission to this user to the logstash index. The new no data component for visualize needs ES data and dataviews to exist.
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
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.
kibana-gis changes LGTM
code review
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.
Tested and all the cases work as expected, LGTM!
- No data / data but no data view cases are handled correctly
- Context of saved vis is not lost after data view creation
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.
Presentation team changes LGTM!
💚 Build SucceededMetrics [docs]Module Count
Async chunks
Page load bundle
Unknown metric groupsasync chunk count
miscellaneous assets size
History
To update your PR or re-run it, just comment with: |
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.
Nice to have this shared :)
Summary
Part of #132146
It implements the no data component to the visualize library.
This component is loaded when:
How to test
Checklist