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 cross-document links to Kibana Alerting docs #779

Merged
merged 4 commits into from
Jun 15, 2021

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Jun 10, 2021

Some recent PRs have moved around some of the alerting docs within the Kibana docs, so some of the links need to be updated.

Relevant PRs:

@ymao1 ymao1 changed the title Updating prerequisites link [Docs] Update links to Kibana Alerting docs Jun 10, 2021
@apmmachine
Copy link
Contributor

apmmachine commented Jun 10, 2021

A documentation preview will be available soon:

@ymao1 ymao1 changed the title [Docs] Update links to Kibana Alerting docs [Docs] Update cross-document links to Kibana Alerting docs Jun 15, 2021
@ymao1 ymao1 marked this pull request as ready for review June 15, 2021 00:43
@ymao1 ymao1 requested a review from a team as a code owner June 15, 2021 00:43
@dedemorton
Copy link
Contributor

@elasticmachine run elasticsearch-ci/docs

@dedemorton
Copy link
Contributor

Running docs build again because it looks like the link was broken when the CI ran originally.

@ymao1 ymao1 self-assigned this Jun 15, 2021
@ymao1 ymao1 added the v7.13.0 label Jun 15, 2021
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.

Tested each link and everything works. Thanks, @ymao1!

I'll handle the backports to 7.x and 7.13.

@ymao1
Copy link
Contributor Author

ymao1 commented Jun 15, 2021

@bmorelli25 Sorry! I put the wrong tag. This only needs backporting to 7.x

@bmorelli25
Copy link
Member

No problem. Thanks!

bmorelli25 added a commit that referenced this pull request Jun 15, 2021
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.

4 participants