Skip to content

[DET-62] Add manual trigger option to Data Observability monitor docs#37570

Open
brkelley wants to merge 3 commits into
masterfrom
bebo.kelley/DET-62-non-scheduled-monitor-docs
Open

[DET-62] Add manual trigger option to Data Observability monitor docs#37570
brkelley wants to merge 3 commits into
masterfrom
bebo.kelley/DET-62-non-scheduled-monitor-docs

Conversation

@brkelley

@brkelley brkelley commented Jun 16, 2026

Copy link
Copy Markdown

What does this PR do? What is the motivation?

Fixes DET-62

Updates the Data Observability monitor documentation to include the Manual scheduling option, which allows monitors to run only when triggered programmatically via the Data Observability APIs.

Merge readiness

  • Ready for merge

AI assistance

Used Claude Code (pi) to create the PR.

@brkelley brkelley requested a review from a team as a code owner June 16, 2026 18:49
@brkelley

Copy link
Copy Markdown
Author

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

Two findings: a grammar error and broken link text on the new "Manual" bullet in the data observability monitor docs, plus an unrelated .vscode/settings.json change that looks like personal IDE config accidentally committed.

Reviewed 13a782b3f6491b62fba6fc36fc11e79b77607d7bworkflow run

Comment thread content/en/monitors/types/data_observability.md Outdated
Comment thread .vscode/settings.json Outdated
@brkelley

Copy link
Copy Markdown
Author

/review

@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

1 blocker on a broken link, 1 nit on URL style.

Reviewed 7fcb4148339ac55e698d0aa79968405adac7f65cworkflow run

Comment thread content/en/monitors/types/data_observability.md Outdated
Comment thread content/en/monitors/types/data_observability.md Outdated
@brkelley brkelley force-pushed the bebo.kelley/DET-62-non-scheduled-monitor-docs branch from 7fcb414 to e785d04 Compare June 16, 2026 19:44
@brkelley

Copy link
Copy Markdown
Author

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

Small, well-scoped addition documenting the new Manual schedule option for Data Observability monitors. Link reference is added correctly. One minor style nit below.

Reviewed e785d049f29b6afd87d1b3f770169d01bb6ccafeworkflow run

Comment thread content/en/monitors/types/data_observability.md Outdated
@cswatt cswatt self-assigned this Jun 16, 2026
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@cswatt cswatt removed their assignment Jun 16, 2026
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.

2 participants