Skip to content

[Docs][Typo] Fix anchor for Wait For Snapshot #53269

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.

Sign up for GitHub

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

Jump to bottom
Merged
jrodewig merged 1 commit into from
Mar 9, 2020
Merged

[Docs][Typo] Fix anchor for Wait For Snapshot #53269

jrodewig merged 1 commit into from
Mar 9, 2020

Conversation

@astik
Copy link
Contributor

@astik astik commented Mar 9, 2020

Previously it pointed to ilm-delete-action but it should point to ilm-wait-for-snapshot-action

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

@astik
Fix anchor for Wait For Snapshot
67c1b43 
Previously it pointed to  ilm-delete-action but it should point to ilm-wait-for-snapshot-action
@jrodewig jrodewig added :Data Management/ILM+SLM Index and Snapshot lifecycle management >docs General docs changes labels Mar 9, 2020
@elasticmachine
Copy link
Collaborator

elasticmachine commented Mar 9, 2020

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

@elasticmachine
Copy link
Collaborator

elasticmachine commented Mar 9, 2020

Pinging @elastic/es-docs (>docs)

@jrodewig
Copy link
Contributor

jrodewig commented Mar 9, 2020

@elasticmachine test this please

jrodewig
jrodewig approved these changes Mar 9, 2020
View reviewed changes
Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @astik.

@jrodewig jrodewig merged commit d1bd395 into elastic:master Mar 9, 2020
jrodewig pushed a commit that referenced this pull request Mar 9, 2020
@astik @jrodewig
[DOCS] Fix "Wait for Snapshot" link in ILM actions reference (#53269)
0065790 
Corrects an anchor for the `Wait for Snapshot` action, which previously linked to the `Delete` action.
jrodewig pushed a commit that referenced this pull request Mar 9, 2020
@astik @jrodewig
[DOCS] Fix "Wait for Snapshot" link in ILM actions reference (#53269)
58a888f 
Corrects an anchor for the `Wait for Snapshot` action, which previously linked to the `Delete` action.
@jrodewig
Copy link
Contributor

jrodewig commented Mar 9, 2020

Backport commits

master d1bd395
7.x 0065790
7.6 58a888f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@jrodewig jrodewig jrodewig approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

:Data Management/ILM+SLM Index and Snapshot lifecycle management >docs General docs changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@astik @elasticmachine @jrodewig