Skip to content

[DOCS] Fix link in Stack Upgrade Guide#1986

Merged
lockewritesdocs merged 2 commits intoelastic:mainfrom
lockewritesdocs:docs__fix-archived-settings-link
Feb 7, 2022
Merged

[DOCS] Fix link in Stack Upgrade Guide#1986
lockewritesdocs merged 2 commits intoelastic:mainfrom
lockewritesdocs:docs__fix-archived-settings-link

Conversation

@lockewritesdocs
Copy link
Copy Markdown
Contributor

@lockewritesdocs lockewritesdocs commented Feb 7, 2022

The link from the Archived settings section of the Stack Upgrade Guide points to the Elasticsearch Upgrade Guide. This link should instead point to the child page for Archived settings.

Preview link: https://stack-docs_1986.docs-preview.app.elstc.co/guide/en/elastic-stack/master/upgrading-elasticsearch.html#archived-settings

Copy link
Copy Markdown
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.

I spotted another update we need to make and left a suggestion.

If we incorporate that, this LGTM.

Co-authored-by: James Rodewig <james.rodewig@elastic.co>
@lockewritesdocs lockewritesdocs merged commit 36c97a6 into elastic:main Feb 7, 2022
@lockewritesdocs lockewritesdocs deleted the docs__fix-archived-settings-link branch February 7, 2022 16:28
lockewritesdocs pushed a commit to lockewritesdocs/stack-docs that referenced this pull request Feb 7, 2022
* [DOCS] Fix link in Stack Upgrade Guide

* Update that settings are archived (and not ignored)

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>
lockewritesdocs pushed a commit to lockewritesdocs/stack-docs that referenced this pull request Feb 7, 2022
* [DOCS] Fix link in Stack Upgrade Guide

* Update that settings are archived (and not ignored)

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>
lockewritesdocs pushed a commit that referenced this pull request Feb 7, 2022
* [DOCS] Fix link in Stack Upgrade Guide

* Update that settings are archived (and not ignored)

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>
lockewritesdocs pushed a commit that referenced this pull request Feb 7, 2022
* [DOCS] Fix link in Stack Upgrade Guide

* Update that settings are archived (and not ignored)

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>

Co-authored-by: James Rodewig <james.rodewig@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants