Skip to content

APPSINTG-356: added miro monitors - #7070

Merged
piyushgupta-sumo merged 3 commits into
mainfrom
APPSINTG-356-miro
Aug 27, 2026
Merged

APPSINTG-356: added miro monitors#7070
piyushgupta-sumo merged 3 commits into
mainfrom
APPSINTG-356-miro

Conversation

@piyushgupta-sumo

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request contains doc related to monitors

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)

APPSINTG-356

@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Aug 26, 2026
Matches the standard alert-description wording used elsewhere in the docs.

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

LGTM — clean addition of a monitors section with correct anchors, plus solid cleanup bundled in (fixed invalid JSON trailing comma, retitled a mismatched query, added null-safe query filters, trimmed filler text). Also applied a wording fix to match the standard "This alert is triggered..." phrasing used elsewhere in the docs.

@piyushgupta-sumo
piyushgupta-sumo added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit 42cf3d1 Aug 27, 2026
4 checks passed
@piyushgupta-sumo
piyushgupta-sumo deleted the APPSINTG-356-miro branch August 27, 2026 09:36
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