Skip to content
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

[Management] Improve messaging with no index patterns #16718

Merged
merged 3 commits into from
Feb 16, 2018

Conversation

chrisronline
Copy link
Contributor

Fixes #16716

Better messaging when there are no visible indices.

screen shot 2018-02-13 at 3 48 20 pm

cc @gchaps for copy verification

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@gchaps
Copy link
Contributor

gchaps commented Feb 13, 2018

Thanks for the clarification offline. Here's my recommendation:

No Elasticsearch indices match your pattern. To view the matching system indices, toggle the switch in the upper right.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@bmcconaghy bmcconaghy left a comment

Choose a reason for hiding this comment

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

LGTM. It would be nice to have this as a common component somewhere. I could use the same thing in index management.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@chrisronline chrisronline merged commit dd7241a into elastic:master Feb 16, 2018
@chrisronline chrisronline deleted the fix/16716 branch February 16, 2018 15:17
chrisronline added a commit to chrisronline/kibana that referenced this pull request Feb 16, 2018
* Improve messaging with no index patterns

* Updated copy

* Adjust messaging
chrisronline added a commit that referenced this pull request Feb 16, 2018
* Improve messaging with no index patterns

* Updated copy

* Adjust messaging
@chrisronline
Copy link
Contributor Author

Backport
6.x: 31b19ed

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

Successfully merging this pull request may close these issues.

5 participants