Skip to content

DOCS-1777 - Document multi-notification support for Scheduled Searches - #6985

Merged
JV0812 merged 9 commits into
mainfrom
DOCS-1777-multi-notification-scheduled-searches
Aug 19, 2026
Merged

DOCS-1777 - Document multi-notification support for Scheduled Searches#6985
JV0812 merged 9 commits into
mainfrom
DOCS-1777-multi-notification-scheduled-searches

Conversation

@JV0812

@JV0812 JV0812 commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request documents the new multi-notification capability for Scheduled Searches: you can now add up to 10 alert types (mixing Email, Webhook, and other alert types) to a single Scheduled Search instead of choosing just one, with each alert configured independently and all sharing the same trigger condition. Updates Step 6 of the "Create a Scheduled Search" procedure and notes that existing single-alert schedules are unaffected.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/DOCS-1777

Scheduled Searches now support adding up to 10 alert types per
search instead of just one, mixing Email, Webhook, and other alert
types with independent per-alert configuration. Update Step 6 of
the create-a-scheduled-search procedure to reflect this and note
that existing single-alert schedules are unaffected.
@JV0812 JV0812 self-assigned this Aug 4, 2026
@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Aug 4, 2026
JV0812 and others added 2 commits August 5, 2026 14:14
…n support

Per SME confirmation (GA, all alert types supported) and a demo
recording, replace the earlier generic Step 6 description with the
actual UI: the Notifications section, Add Notification link, and
collapsed notification summary rows, plus screenshots of adding a
second alert and the finished multi-alert list.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@JV0812
JV0812 marked this pull request as ready for review August 10, 2026 11:57
@ashutosh-kumbhaj-sumo

ashutosh-kumbhaj-sumo commented Aug 13, 2026

Copy link
Copy Markdown

Suggestions :-

  1. "Alert types" vs "notifications" terminology: The blog post says "up to 10 alert types" which could be misread as 10 distinct types. The docs clarify you can add "multiple Email alerts with different recipients" which is good — but the blog post could benefit from saying "up to 10 notifications" instead of "10 alert types" to avoid confusion.
  2. The feature introduces a new notifications array in the API (Content Sync / Log Searches API). There's no mention of the API changes in this docs PR. Does API docs live elsewhere? or are they not required?

Rest looks good to me.

"10 alert types" could be misread as 10 distinct types; each alert is a notification, and you can add multiples of the same type.
@JV0812

JV0812 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks for the review, @ashutosh-kumbhaj-sumo!

  1. Fixed — the release note now says "up to 10 notifications" instead of "10 alert types" to avoid the distinct-types misread (d4f12f182).
  2. On the API doc question: no change needed here. docs/api/log-searches.md is a landing/reference page (region-endpoint table + role capabilities) that links out to the auto-generated Swagger/OpenAPI spec — it's never had field-level schema details for any past feature (checked its history back through prior alert-type additions). The new notifications array will surface automatically in the Swagger reference once the API spec is updated on the engineering side; that's outside this docs PR's scope.

@amee-sumo amee-sumo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clean, well-scoped update. Verified the anchor rename (#step-6-choose-a-scheduled-search-alert-type → #step-6-add-scheduled-search-alert-types) doesn't break any other cross-references in the repo, the Step 5 cross-link resolves correctly, and the release note is consistent with the doc content.

One thing worth double-checking before merge: the sentence introducing the second screenshot says "combines Email, Webhook, and ServiceNow Connection alerts" (3 types), but the image's alt text says "listing four configured notifications." Could be accurate (e.g. two Email alerts + Webhook + ServiceNow = 4 notifications, 3 types), but worth a quick visual check against the actual screenshot to confirm the count/description matches.

Approving.

JV0812 and others added 2 commits August 17, 2026 15:25
…ption

The screenshot shows 4 notifications (two Email, one Webhook, one ServiceNow Connection), not 4 distinct types. Update the intro sentence and alt text to match.
@JV0812

JV0812 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator Author

@amee-sumo Good catch — checked the actual screenshot, and it's 4 notifications total: Email, Webhook, ServiceNow Connection, Email (Email appears twice). So the "3 types" phrasing and the alt text's "different alert types" were both slightly off. Fixed (22cb50a99): the sentence now says "combines four notifications ... two Email alerts, a Webhook alert, and a ServiceNow Connection alert," and the alt text matches. This also nicely reinforces the "multiple of the same alert type" point from Step 6.

@JV0812
JV0812 enabled auto-merge August 19, 2026 15:49
@JV0812
JV0812 added this pull request to the merge queue Aug 19, 2026
Merged via the queue into main with commit 54fd8cc Aug 19, 2026
4 checks passed
@JV0812
JV0812 deleted the DOCS-1777-multi-notification-scheduled-searches branch August 19, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants