Skip to content

[DOCS] Update invalidate api key doc for the new ids field#63860

Merged
ywangd merged 1 commit intoelastic:masterfrom
ywangd:es-63224-follow-up-doc-update
Oct 21, 2020
Merged

[DOCS] Update invalidate api key doc for the new ids field#63860
ywangd merged 1 commit intoelastic:masterfrom
ywangd:es-63224-follow-up-doc-update

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Oct 19, 2020

This PR is a follow-up for #63224 and adds the missing doc update for the new ids field.

Preview

https://elasticsearch_63860.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/security-api-invalidate-api-key.html

@ywangd ywangd added >docs General docs changes :Security/Security Security issues without another label v8.0.0 v7.10.0 labels Oct 19, 2020
@ywangd ywangd requested a review from tvernum October 19, 2020 00:45
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-security (:Security/Security)

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added Team:Docs Meta label for docs team Team:Security Meta label for security team labels Oct 19, 2020
@lcawl
Copy link
Copy Markdown
Contributor

lcawl commented Oct 19, 2020

@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Oct 19, 2020

https://github.com/elastic/elasticsearch/blob/master/docs/java-rest/high-level/security/invalidate-api-key.asciidoc

Thanks for the review @lcawl The HLRC doc didn't get updated because the change is not made for the HLRC code yet. This PR is a follow-up and targets the 7.10 release which is feature freezed already. Hence it is intentionally kept minimum without changes to HLRC. Please let me know if you think otherwise. Thanks!

@lcawl lcawl changed the title Update invalidate api key doc for the new ids field [DOCS] Update invalidate api key doc for the new ids field Oct 19, 2020
Copy link
Copy Markdown
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM

@ywangd ywangd added the v7.11.0 label Oct 20, 2020
@ywangd ywangd merged commit c41f87c into elastic:master Oct 21, 2020
ywangd added a commit to ywangd/elasticsearch that referenced this pull request Oct 21, 2020
Follow-up for elastic#63224 and adds the missing doc update for the new ids field.
ywangd added a commit to ywangd/elasticsearch that referenced this pull request Oct 21, 2020
Follow-up for elastic#63224 and adds the missing doc update for the new ids field.
ywangd added a commit that referenced this pull request Oct 21, 2020
Follow-up for #63224 and adds the missing doc update for the new ids field.
ywangd added a commit that referenced this pull request Oct 21, 2020
Follow-up for #63224 and adds the missing doc update for the new ids field.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes :Security/Security Security issues without another label Team:Docs Meta label for docs team Team:Security Meta label for security team v7.10.0 v7.11.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants