Skip to content

SUMO-28841 | Apoorv | Add. monitors and new screenshots - Amazon GuardDuty #6907

Merged
Apoorvkudesia-sumologic merged 2 commits into
mainfrom
AmazonGaurdDutyUpdate
Jul 17, 2026
Merged

SUMO-28841 | Apoorv | Add. monitors and new screenshots - Amazon GuardDuty #6907
Apoorvkudesia-sumologic merged 2 commits into
mainfrom
AmazonGaurdDutyUpdate

Conversation

@Apoorvkudesia-sumologic

@Apoorvkudesia-sumologic Apoorvkudesia-sumologic commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request updates Amazon GuardDuty docs with new monitors and updation of stale screenshots

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)

@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Jul 15, 2026
Align the "Installing the Amazon GuardDuty app" heading casing with the
other section headings.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@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.

Approving. This adds a Create monitors section (4 alerts) plus Upgrade/Downgrade + Uninstalling sections, and refreshes 4 dashboard screenshots.

Verified:

  • All 4 new images exist with valid sizes and correct references; the 4 deleted images are referenced nowhere else in the repo, so nothing breaks.
  • Monitors follow conventions — "This alert is triggered" wording, Count < = 0 operator format, consistent "Amazon GuardDuty -" naming; no duplicate headings; reuse imports resolve.

I pushed one small follow-up commit (d8a4b14): lowercased "app" in the "Installing the Amazon GuardDuty app" heading to match the other section headings (was capital "App").

Minor items to confirm before merge (non-blocking): the title ticket SUMO-28841 may be missing a digit, and the PR body has no "type of change" checkbox selected or ticket filled in.

— via Claude Code

@Apoorvkudesia-sumologic
Apoorvkudesia-sumologic added this pull request to the merge queue Jul 17, 2026
Merged via the queue into main with commit b1a0753 Jul 17, 2026
5 checks passed
@Apoorvkudesia-sumologic
Apoorvkudesia-sumologic deleted the AmazonGaurdDutyUpdate branch July 17, 2026 14:18
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.

2 participants