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] Create rule updates 8.4 #2339

Merged
merged 7 commits into from
Aug 24, 2022
Merged

[DOCS] Create rule updates 8.4 #2339

merged 7 commits into from
Aug 24, 2022

Conversation

@jmikell821 jmikell821 added Team: Detections/Response Detections and Response readyforQA PRs that are ready for QA review. Feature: Rules v8.4.0 labels Aug 23, 2022
@jmikell821 jmikell821 self-assigned this Aug 23, 2022
@ghost
Copy link

ghost commented Aug 23, 2022

Hi @jmikell821 ,

The preview link for above changes is not available in this PR.
However, We have manage to get the preview link here and reviewed only this page.

We have found 2 issues.

Please let us know if there is any other preview link that we need to review.

Thanks!!

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.

Looks good, left a few suggestions

docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Show resolved Hide resolved
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.

Just a handful of minor edits. Otherwise, LGTM!

docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
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.

Lots of good updates here! I added a few revision suggestions.

I also suggest cropping the inline icon image saved-query-menu.png to leave just 1 pixel padding on each side, and no border. With all the extra padding, the icon itself ends up looking very small and hard to see.

image vs. image

docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
docs/detections/rules-ui-create.asciidoc Outdated Show resolved Hide resolved
@jmikell821 jmikell821 merged commit a2a21e7 into main Aug 24, 2022
mergify bot pushed a commit that referenced this pull request Aug 24, 2022
* Publishing branch for a quick save.

* Saving current edits.

* Slight edits.

* Addressing two quick bugs.

* Adding feedback.

(cherry picked from commit a2a21e7)
@jmikell821 jmikell821 mentioned this pull request Aug 24, 2022
20 tasks
jmikell821 added a commit that referenced this pull request Aug 24, 2022
* Publishing branch for a quick save.

* Saving current edits.

* Slight edits.

* Addressing two quick bugs.

* Adding feedback.

(cherry picked from commit a2a21e7)

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
acorretti pushed a commit that referenced this pull request Nov 19, 2024
* Publishing branch for a quick save.

* Saving current edits.

* Slight edits.

* Addressing two quick bugs.

* Adding feedback.

(cherry picked from commit a2a21e7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment