Skip to content

Conversation

@joepeeples
Copy link
Contributor

@joepeeples joepeeples commented Aug 8, 2022

Resolves #2262.

Previews:

@joepeeples joepeeples marked this pull request as ready for review August 11, 2022 20:14
Copy link
Contributor

@nastasha-solomon nastasha-solomon left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @joepeeples !

@mergify

This comment was marked as resolved.

@joepeeples joepeeples added the readyforQA PRs that are ready for QA review. label Aug 16, 2022
@ghost
Copy link

ghost commented Aug 17, 2022

Hi @joepeeples,

We tested this PR and found 01 issue.

We will Re-validate this PR once the above issue is fixed.

Thanks!

@ghost ghost removed the readyforQA PRs that are ready for QA review. label Aug 17, 2022
security-docs#2310
@joepeeples
Copy link
Contributor Author

@ghost
Copy link

ghost commented Aug 18, 2022

Hi @joepeeples,

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

image

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

Thanks!!

@ghost ghost added the QA:Validated Issue has been Validated by QA Team label Aug 18, 2022
@joepeeples joepeeples mentioned this pull request Aug 18, 2022
20 tasks
Copy link
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify

This comment was marked as resolved.

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.

Quick and easy -- LGTM! 👍

@joepeeples joepeeples merged commit c3a9494 into main Aug 23, 2022
mergify bot pushed a commit that referenced this pull request Aug 23, 2022
* Update text

* Update and resize screenshots

* Update screenshots of Rules page

Renamed button appears on Rules page

* Correction (QA)

security-docs#2310

* Trim cross-reference

(cherry picked from commit c3a9494)
joepeeples added a commit that referenced this pull request Aug 23, 2022
* Update text

* Update and resize screenshots

* Update screenshots of Rules page

Renamed button appears on Rules page

* Correction (QA)

security-docs#2310

* Trim cross-reference

(cherry picked from commit c3a9494)

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
@joepeeples joepeeples deleted the issue-2262-import-value-lists branch August 23, 2022 15:08
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.

[DOCS] Import value lists: renamed button, new flyout UI

5 participants