Skip to content

Conversation

DianaIgnatescu
Copy link
Contributor

No description provided.

Copy link

netlify bot commented Sep 18, 2025

Deploy Preview for snowplow-docs ready!

Name Link
🔨 Latest commit 7158823
🔍 Latest deploy log https://app.netlify.com/projects/snowplow-docs/deploys/68cd846ed2b2070008bd1bda
😎 Deploy Preview https://deploy-preview-1414--snowplow-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@DianaIgnatescu DianaIgnatescu added the do not merge Flag to denote a Issue or PR which should not yet be merged (usually pending a release) label Sep 18, 2025
Copy link
Contributor

@gibbok-snowplow gibbok-snowplow left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, few suggestions:

  • rename the image with full name, so instead of dq please consider data_quality, this is better for SEO
  • I would add few uses cases describing the value

Copy link
Contributor

@jborlase-snowplow jborlase-snowplow left a comment

Choose a reason for hiding this comment

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

Small nitpick, but otherwise I think it looks good

Choose from the available trigger options:

- **When above value**: Set an absolute threshold for failed events (e.g., 1,500 failed events per hour)
- **On any issue**: Alert immediately when any failed events are detected
Copy link
Contributor

Choose a reason for hiding this comment

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

"Immediately" makes it sound like it overrides the delivery window - we could remove it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Flag to denote a Issue or PR which should not yet be merged (usually pending a release)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants