Skip to content
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

[docs] Update link for "ccr-overview-beats" anchor #2589

Merged
merged 2 commits into from
Sep 18, 2019
Merged

[docs] Update link for "ccr-overview-beats" anchor #2589

merged 2 commits into from
Sep 18, 2019

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Aug 19, 2019

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.

Dependencies

Don't merge this PR until the following PRs are merged:

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
@bmorelli25
Copy link
Member

Thanks James! Are you only moving the documentation over in master? Or will this need to be backported?

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

@jrodewig
Copy link
Contributor Author

@bmorelli25 Thanks! I'm planning to backport to 7.x and v7.3. Let me know the best way to go about that.

@bmorelli25
Copy link
Member

Ok, great. I can handle the backports if that's easier for you!

@jrodewig
Copy link
Contributor Author

Sounds good. I'll give you a ping when it's ready for a merge. Thanks again @bmorelli25!

@bmorelli25 bmorelli25 added this to the 7.5 milestone Sep 3, 2019
@jrodewig jrodewig marked this pull request as ready for review September 17, 2019 20:28
@jrodewig
Copy link
Contributor Author

@bmorelli25 This is ready for merge at your convenience. The change will need to be backported to 7.x, 7.4, and 7.3. Thanks!

@bmorelli25
Copy link
Member

Thanks @jrodewig. I'll merge and handle the backports.

@bmorelli25 bmorelli25 merged commit 88f3ec6 into elastic:master Sep 18, 2019
bmorelli25 pushed a commit to bmorelli25/apm-server that referenced this pull request Sep 18, 2019
bmorelli25 pushed a commit to bmorelli25/apm-server that referenced this pull request Sep 18, 2019
bmorelli25 pushed a commit to bmorelli25/apm-server that referenced this pull request Sep 18, 2019
@bmorelli25 bmorelli25 self-assigned this Sep 18, 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.

2 participants