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

Update docs and image to match new index name pattern #228

Merged
merged 2 commits into from
Sep 4, 2015

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Sep 4, 2015

As in #186 the index pattern was updated, the screenshot and additional parts of the documentation had to be updated.

@ruflin ruflin added the review label Sep 4, 2015
@tsg
Copy link
Contributor

tsg commented Sep 4, 2015

The screenshot it is a bit big. It's good to have the images max 800px in the docs. Here is how it currently looks:

screen shot 2015-09-04 at 09 46 39

@tsg
Copy link
Contributor

tsg commented Sep 4, 2015

We should do the same for the Topbeat guide.

@ruflin
Copy link
Contributor Author

ruflin commented Sep 4, 2015

Image exchanged. I will also do it for Topbeat: https://github.com/elastic/topbeat/issues/25

tsg added a commit that referenced this pull request Sep 4, 2015
Update docs and image to match new index name pattern
@tsg tsg merged commit 7f92351 into elastic:master Sep 4, 2015
@tsg
Copy link
Contributor

tsg commented Sep 4, 2015

Thanks!

ruflin added a commit that referenced this pull request Dec 2, 2015
* Check for errors in new line first
* Refactor logging to make it more useful. See #228
* Remove obsolete .gitignore tags
* Make scan_frequency configurable in tests.
* Fix flaky test through increasing scan_frequency. This prevents the race condition
@ruflin ruflin deleted the doc-screenshot-update branch December 2, 2015 12:07
tsg added a commit to tsg/beats that referenced this pull request Jan 20, 2016
Update docs and image to match new index name pattern
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.

2 participants