Skip to content

[DOCS] Update anchor and add redirect for aliases #77349

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.

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

Merged
merged 1 commit into from
Sep 7, 2021
Merged

[DOCS] Update anchor and add redirect for aliases #77349

merged 1 commit into from
Sep 7, 2021

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Sep 7, 2021

PRs #73062 and #73043 repurposed the alias anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a /current/ link to the alias page.

This updates the anchor for the alias guide and adds a redirect page to
disambiguate the alias anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.

Closes #77034.

PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This updates the anchor for the alias guide and adds a redirect page to
disambiguate the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.

Closes #77034.
[role="exclude",id="alias"]
=== Aliases

For field aliases, refer to <<field-alias>>.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In 7.13 and 7.12, I'll only include this portion of the redirect.

@jrodewig jrodewig marked this pull request as ready for review September 7, 2021 12:54
@jrodewig jrodewig added the >docs General docs changes label Sep 7, 2021
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Sep 7, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Tested the output, and it's looking great. 👍 thanks for tackling this one @jrodewig!

@jrodewig
Copy link
Contributor Author

jrodewig commented Sep 7, 2021

Thanks @lockewritesdocs!

@jrodewig jrodewig merged commit cfae697 into elastic:master Sep 7, 2021
@jrodewig jrodewig deleted the docs__alias-redirect branch September 7, 2021 13:42
jrodewig added a commit that referenced this pull request Sep 7, 2021
PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This adds a redirect page for the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.
jrodewig added a commit that referenced this pull request Sep 7, 2021
PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This adds a redirect page for the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.
elasticsearchmachine pushed a commit that referenced this pull request Sep 7, 2021
…7357)

* [DOCS] Update anchor and add redirect for aliases (#77349)

PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This updates the anchor for the alias guide and adds a redirect page to
disambiguate the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.

Closes #77034.
# Conflicts:
#	docs/reference/search/multi-search.asciidoc

* [DOCS] Add anchor text to field alias redirect
elasticsearchmachine pushed a commit that referenced this pull request Sep 7, 2021
…7356)

* [DOCS] Update anchor and add redirect for aliases (#77349)

PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This updates the anchor for the alias guide and adds a redirect page to
disambiguate the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.

Closes #77034.
# Conflicts:
#	docs/reference/search/multi-search.asciidoc

* [DOCS] Add anchor text to field alias redirect
elasticsearchmachine pushed a commit that referenced this pull request Sep 7, 2021
)

* [DOCS] Update anchor and add redirect for aliases (#77349)

PRs #73062 and #73043 repurposed the `alias` anchor for a new guide for index
and data stream aliases. Previously, this anchor was used for our field alias
documentation.

Repurposing the anchor has caused continuity errors for users selecting
different versions of the ES docs. It could also cause confusion for users with
a `/current/` link to the `alias` page.

This updates the anchor for the alias guide and adds a redirect page to
disambiguate the `alias` anchor.

It also fixes a bread crumb issue for redirects following the 'Modifying your
Data' redirect page.

Closes #77034.
# Conflicts:
#	docs/reference/search/multi-search.asciidoc

* [DOCS] Add anchor text to field alias redirect
@jakelandis jakelandis removed the v8.0.0 label Sep 15, 2021
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.

Field Alias datatype link to current docs incorrect
5 participants