Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

vkumbha
Copy link
Contributor

@vkumbha vkumbha commented Jul 2, 2025

No description provided.

@vkumbha vkumbha requested a review from rajlearner17 July 2, 2025 10:27
@vkumbha vkumbha self-assigned this Jul 2, 2025
Copy link

github-actions bot commented Jul 2, 2025

Preview Available 🚀

Commit Author: David Boeke
Commit Message: update lw09 images for wednesday. (#925)

Preview Link: turbot-com-git-docs-guardrailsissue-349-add-updat-586e2a-turbot.vercel.app

@vkumbha vkumbha linked an issue Jul 2, 2025 that may be closed by this pull request
@rajlearner17 rajlearner17 requested a review from Copilot July 2, 2025 10:38
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces support for a Custom Webhook notification channel by updating documentation, adding new template examples, and adjusting filter rules.

  • Added Nunjucks examples for custom webhook action and control templates.
  • Updated the notifications index to list and explain the webhook channel and its policies.
  • Added “Custom Webhook” to the filter rules documentation.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
docs/guides/using-guardrails/notifications/templates.md Added example custom webhook control and action templates in Nunjucks.
docs/guides/using-guardrails/notifications/index.md Updated channels list and template configuration to include Webhook.
docs/guides/using-guardrails/notifications/filter-rules.md Added Custom Webhook to the list of supported filter rule channels.
Comments suppressed due to low confidence (2)

docs/guides/using-guardrails/notifications/filter-rules.md:16

  • [nitpick] Channel naming is inconsistent across docs; consider aligning this label with “Custom webhook notifications” used in index.md for consistency.
- Custom Webhook

docs/guides/using-guardrails/notifications/filter-rules.md:16

  • Include an example snippet showing how to configure webhookUrl in a rule-set (similar to index.md) so users can see how to set up Custom Webhook rules.
- Custom Webhook

Copy link
Contributor

@rajlearner17 rajlearner17 left a comment

Choose a reason for hiding this comment

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

LGTM

@rajlearner17 rajlearner17 merged commit 95dfa66 into main Jul 2, 2025
4 checks passed
@rajlearner17 rajlearner17 deleted the issue-349-add-update-notification-guide branch July 2, 2025 10:48
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.

Add guide - Add / Update Notification Guide
2 participants