Skip to content

Remove included docs#50557

Merged
DaveCTurner merged 4 commits intoelastic:7.xfrom
DaveCTurner:2020-01-02-fix-cat-nodes-local-param-docs
Jan 2, 2020
Merged

Remove included docs#50557
DaveCTurner merged 4 commits intoelastic:7.xfrom
DaveCTurner:2020-01-02-fix-cat-nodes-local-param-docs

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

In #50499 we accidentally duplicated the docs for the ?local parameter to the
GET _cat/nodes API. This commit removes the duplicate docs.

In elastic#50499 we accidentally duplicated the docs for the `?local` parameter to the
`GET _cat/nodes` API. This commit removes the duplicate docs.
@DaveCTurner DaveCTurner added >docs General docs changes :Core/Infra/Core Core issues without another label v7.6.0 labels Jan 2, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Core)

Copy link
Copy Markdown
Contributor

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

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

LGTM :)

deprecated:[7.6.0,This parameter does not cause this API to act locally. It will be removed in version 8.0.]
If `true`, the request computes the list of selected nodes from the local
cluster state. Defaults to `false`, which means the list of selected nodes is
computed from the cluster state on the master node. In either case the
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

While we're at it: needs a , after "In either case" doesn't it? :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, I think it's acceptable (and better) as it is.

@DaveCTurner DaveCTurner merged commit b209e7b into elastic:7.x Jan 2, 2020
@DaveCTurner DaveCTurner deleted the 2020-01-02-fix-cat-nodes-local-param-docs branch January 2, 2020 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >docs General docs changes v7.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants