Skip to content

Conversation

@nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented Aug 3, 2022

Addresses #2227.

Preview here.

@nastasha-solomon nastasha-solomon requested a review from e40pud August 4, 2022 15:21
@nastasha-solomon nastasha-solomon marked this pull request as ready for review August 12, 2022 18:26
Co-authored-by: Marshall Main <55718608+marshallmain@users.noreply.github.com>
Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

LGTM! A couple very minor edit suggestions.

nastasha-solomon and others added 3 commits August 16, 2022 16:08
Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
@nastasha-solomon nastasha-solomon added the readyforQA PRs that are ready for QA review. label Aug 16, 2022
@ghost
Copy link

ghost commented Aug 17, 2022

Hi @nastasha-solomon,

We have reviewed the shared preview link and observed that all changes are present 🟢

Screenshots:

image

Hence we are adding the "QA: Validated" label to it.

Thanks!!

@ghost ghost added QA:Validated Issue has been Validated by QA Team and removed readyforQA PRs that are ready for QA review. labels Aug 17, 2022
Copy link
Contributor

@jmikell821 jmikell821 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for getting this in! 👍

@nastasha-solomon nastasha-solomon merged commit 10fbdd2 into main Aug 17, 2022
mergify bot pushed a commit that referenced this pull request Aug 17, 2022
Co-authored-by: Marshall Main <55718608+marshallmain@users.noreply.github.com>
Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
(cherry picked from commit 10fbdd2)
nastasha-solomon added a commit to nastasha-solomon/security-docs that referenced this pull request Sep 19, 2022
…lastic#2251) (elastic#2313)

Co-authored-by: Marshall Main <55718608+marshallmain@users.noreply.github.com>
Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
Co-authored-by: nastasha-solomon <79124755+nastasha-solomon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature: Rules QA:Validated Issue has been Validated by QA Team Team: Detections/Response Detections and Response Team: Docs v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants