Skip to content

Conversation

@nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented Oct 15, 2025

Fixes #2110

The UI has copy and visual cues that specify the limit of fields that users can specify in the Group by field, so mentioning that info in the docs is not necessary. I did however expand the note under the step 2c to include the information in this comment.

Preview - See the note under step 2c

@nastasha-solomon nastasha-solomon self-assigned this Oct 15, 2025
@github-actions
Copy link

github-actions bot commented Oct 15, 2025

🔍 Preview links for changed docs

@nastasha-solomon nastasha-solomon marked this pull request as ready for review October 16, 2025 19:31
@nastasha-solomon nastasha-solomon requested a review from a team as a code owner October 16, 2025 19:31
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

Copy link

@denar50 denar50 left a comment

Choose a reason for hiding this comment

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

LGTM!

@nastasha-solomon nastasha-solomon merged commit b589838 into main Oct 17, 2025
6 of 7 checks passed
@nastasha-solomon nastasha-solomon deleted the issue-2110-threshold-rule-grouping branch October 17, 2025 13:31
rhr323 pushed a commit to rhr323/docs-content that referenced this pull request Oct 27, 2025
…ation (elastic#3492)

Fixes elastic#2110

The UI has copy and visual cues that specify the limit of fields that
users can specify in the **Group by** field, so mentioning that info in
the docs is not necessary. I did however expand the note under the step
2c to include the information [in this
comment](elastic/security-team#8240 (comment)).


[Preview](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/3492/solutions/security/detect-and-alert/create-detection-rule#create-threshold-rule)
- See the note under step 2c
naemono pushed a commit to naemono/docs-content that referenced this pull request Oct 28, 2025
…ation (elastic#3492)

Fixes elastic#2110

The UI has copy and visual cues that specify the limit of fields that
users can specify in the **Group by** field, so mentioning that info in
the docs is not necessary. I did however expand the note under the step
2c to include the information [in this
comment](elastic/security-team#8240 (comment)).


[Preview](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/3492/solutions/security/detect-and-alert/create-detection-rule#create-threshold-rule)
- See the note under step 2c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Internal]: improvements to threshold rule documentation

4 participants