Skip to content

[DOCS] Reformat cat count API #45160

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

Merged
merged 1 commit into from
Aug 6, 2019
Merged

[DOCS] Reformat cat count API #45160

merged 1 commit into from
Aug 6, 2019

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Aug 2, 2019

Relates to elastic/docs#937.

This PR updates the cat count API to align with the new API reference template.

Dependent on #45119 and #45158, which add several shared parameters to the common-parms.asciidoc file.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@jrodewig jrodewig force-pushed the reformat-cat-count branch from f22f6b8 to 8ea8a30 Compare August 2, 2019 21:02
@jrodewig jrodewig force-pushed the reformat-cat-count branch from 8ea8a30 to 85d9a97 Compare August 2, 2019 21:10
@jrodewig jrodewig force-pushed the reformat-cat-count branch from 85d9a97 to 93de718 Compare August 5, 2019 15:27
@jrodewig jrodewig marked this pull request as ready for review August 5, 2019 15:28
@jrodewig jrodewig force-pushed the reformat-cat-count branch from 93de718 to 6be9bb4 Compare August 5, 2019 16:38
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM

@jrodewig jrodewig merged commit 5bf58c2 into master Aug 6, 2019
@jrodewig jrodewig deleted the reformat-cat-count branch August 6, 2019 12:36
jrodewig added a commit that referenced this pull request Aug 6, 2019
jrodewig added a commit that referenced this pull request Aug 6, 2019
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.

4 participants