Skip to content

[DOCS] Fixes link to role mapping#2775

Merged
bmorelli25 merged 2 commits intoelastic:masterfrom
lcawl:security-links
Oct 28, 2019
Merged

[DOCS] Fixes link to role mapping#2775
bmorelli25 merged 2 commits intoelastic:masterfrom
lcawl:security-links

Conversation

@lcawl
Copy link
Copy Markdown
Member

@lcawl lcawl commented Oct 4, 2019

Related to elastic/elasticsearch#46880

This PR fixes a broken link to the security content in the Stack Overview, which has moved here:
https://www.elastic.co/guide/en/elasticsearch/reference/master/mapping-roles.html#mapping-roles-file

@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 4, 2019

Codecov Report

Merging #2775 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2775   +/-   ##
=======================================
  Coverage   79.15%   79.15%           
=======================================
  Files          83       83           
  Lines        4332     4332           
=======================================
  Hits         3429     3429           
  Misses        903      903

@lcawl lcawl requested a review from bmorelli25 October 17, 2019 01:53
@lcawl lcawl marked this pull request as ready for review October 17, 2019 01:54
Copy link
Copy Markdown
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

Thanks Lisa!

@bmorelli25 bmorelli25 merged commit 3a2711b into elastic:master Oct 28, 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.

3 participants