pipedream
pipedream copied to clipboard
[BUG] Datadog - New Monitor Event Trigger async options broken #3459
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated |
|---|---|---|---|
| pipedream-docs | ✅ Ready (Inspect) | Visit Preview | Jul 29, 2022 at 5:27PM (UTC) |
| pipedream-docs-redirect-do-not-edit | ✅ Ready (Inspect) | Visit Preview | Jul 29, 2022 at 5:27PM (UTC) |
Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:
- [ ] Create components to address specific use cases whenever possible
- [ ] Component
keys should follow the formatapp_name_slug-slugified-component-name - [ ] Components should follow the standard directory structure
- [ ] Prefer Node.js client libraries to REST APIs
- [ ] When making API requests, handle pagination to ensure all data / events are processed
- [ ] Use
secretprops to capture sensitive data - [ ] Props and methods should be defined in app files whenever possible
- [ ] Document methods with JS Docs
- [ ] Use
optionalprops whenever possible, and set adefaultvalue where you can - [ ] Use async options to accept user input wherever possible
- [ ] Create subscription webhooks sources (vs polling sources) whenever possible
- [ ] Always emit a
summarywith events - [ ] Use a built-in deduping strategy wherever possible
- [ ] When building webhook sources, follow these best practices
- [ ] When building polling sources, follow these best practices
@lcaresia This fix worked when I tested initially but after adding more monitors to my datadog account the same issue was observed.

Verification Completed, This is ready for release.