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][Alerting] updates images in Alerts Management and adds docs for the Run When field #92225

Merged
merged 6 commits into from
Feb 23, 2021

Conversation

gmmorris
Copy link
Contributor

@gmmorris gmmorris commented Feb 22, 2021

Summary

Addresses #90007

Adding missing docs for #82472

alert-flyout-action-variables

alert-flyout-action-details

Preview: https://kibana_92225.docs-preview.app.elstc.co/guide/en/kibana/master/defining-alerts.html

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@gmmorris gmmorris requested a review from a team as a code owner February 22, 2021 16:27
@gmmorris gmmorris changed the title updated images and added docs for Run When [Docs][Alerting] updates images in Alerts Management and adds docs for the Run When field Feb 22, 2021
@gmmorris gmmorris added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.11.2 v7.12.0 v8.0.0 labels Feb 22, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@gmmorris gmmorris added the release_note:skip Skip the PR/issue when compiling release notes label Feb 22, 2021
Copy link
Contributor

@YulNaumenko YulNaumenko left a comment

Choose a reason for hiding this comment

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

LGTM! Also +1 on adding a screenshot, because it's difficult to find in UI at the first time :) Please add @gchaps

Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gmmorris gmmorris requested a review from gchaps February 22, 2021 17:01
Copy link

@akashgp09 akashgp09 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

docs/user/alerting/defining-alerts.asciidoc Outdated Show resolved Hide resolved
docs/user/alerting/defining-alerts.asciidoc Outdated Show resolved Hide resolved
docs/user/alerting/defining-alerts.asciidoc Outdated Show resolved Hide resolved
docs/user/alerting/defining-alerts.asciidoc Outdated Show resolved Hide resolved
gmmorris and others added 4 commits February 23, 2021 10:58
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
@gmmorris gmmorris merged commit 4c82ffc into elastic:master Feb 23, 2021
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 23, 2021
…r the Run When field (elastic#92225)

Updates images in Alerts Management Docs and usage of the Run When field
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 23, 2021
…r the Run When field (elastic#92225)

Updates images in Alerts Management Docs and usage of the Run When field
gmmorris added a commit that referenced this pull request Feb 23, 2021
…r the Run When field (#92225) (#92397)

Updates images in Alerts Management Docs and usage of the Run When field
gmmorris added a commit that referenced this pull request Feb 23, 2021
…r the Run When field (#92225) (#92398)

Updates images in Alerts Management Docs and usage of the Run When field
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.11.2 v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants