Skip to content

Fix #22510, dashboard-only mode doesn't display saved searches#22685

Merged
chrisdavies merged 1 commit intoelastic:masterfrom
chrisdavies:dashboard/dashboard-only-mode-22510
Sep 5, 2018
Merged

Fix #22510, dashboard-only mode doesn't display saved searches#22685
chrisdavies merged 1 commit intoelastic:masterfrom
chrisdavies:dashboard/dashboard-only-mode-22510

Conversation

@chrisdavies
Copy link
Copy Markdown
Contributor

Add a dependency on ui/pager to the dashboard-only UI.

Copy link
Copy Markdown

@stacey-gammon stacey-gammon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM code review only.

Talked to Chris offline and we will work on adding more tests to cover this separately so we can get this fix in for 6.4.1.

cc @elastic/kibana-qa - to test, just log in as a dashboard only mode user, with access to the data, and try to view a dashboard with a saved search on it. Saved search before would come up blank with a console error, now with this fix, should show up with data.

@stacey-gammon
Copy link
Copy Markdown

Fixes #22510

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@chrisdavies
Copy link
Copy Markdown
Contributor Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@chrisdavies
Copy link
Copy Markdown
Contributor Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants