Skip to content

docs(usage): document notification triggers and asset-scoped filters (#6937) - #7680

Open
Abdullah (AzazelSensei) wants to merge 1 commit into
OpenAEV-Platform:mainfrom
AzazelSensei:docs-6937-notification-triggers
Open

docs(usage): document notification triggers and asset-scoped filters (#6937)#7680
Abdullah (AzazelSensei) wants to merge 1 commit into
OpenAEV-Platform:mainfrom
AzazelSensei:docs-6937-notification-triggers

Conversation

@AzazelSensei

Copy link
Copy Markdown

Proposed changes

  • Rewrote the notifications usage guide for live and digest triggers, resource types, event types (including Scenario score degradation), filters, and UI / email / webhook notifiers.
  • Updated the findings guide so assets are every inventory category except security platforms, and the overview uses Impacted assets.

Testing Instructions

  1. Open the docs preview for docs/docs/usage/notifications.md and docs/docs/usage/evaluate/findings/findings.md.
  2. Check the trigger and notifier steps against Profile > Triggers and Settings > Customization > Notifiers.

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Rewrite the notifications guide for live and digest triggers, resource
types, events, filters, and notifiers. Findings docs now say assets
cover every inventory category except security platforms.
Copilot AI lite review requested due to automatic review settings August 31, 2026 07:10
@filigran-cla-bot

filigran-cla-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

Contributor License Agreement

CLA signed 💚

Thank you Abdullah (@AzazelSensei) for signing the Contributor License Agreement! Your pull request can now be reviewed and merged.

We appreciate your contribution to Filigran's open source projects! ❤️

This is an automated message from the Filigran CLA Bot.

@filigran-cla-bot filigran-cla-bot Bot added the cla:pending CLA signature required. label Aug 31, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution. This PR is but one step away from being ready for merging: all commits must be PGP-signed. To get started, please see https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@Filigran-Automation Filigran Automation (Filigran-Automation) added the community Contribution from the community. label Aug 31, 2026
@Filigran-Automation Filigran Automation (Filigran-Automation) changed the title docs(usage): document notification triggers and asset-scoped filters #6937 docs(usage): document notification triggers and asset-scoped filters (#6937) Aug 31, 2026

Copilot AI 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.

Pull request overview

This PR updates the user-facing documentation to reflect the current Notifications (triggers + notifiers, live + digest) and clarifies that Findings and related filters are asset-scoped across all inventory categories except security platforms.

Changes:

  • Rewrites the Notifications usage guide to explain triggers (live/digest), resource/event types, filters, and notifiers (UI/email/webhook).
  • Updates the Findings guide to clarify “Impacted assets” terminology and that findings can attach to assets across the unified inventory (excluding security platforms).

Reviewed changes

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

File Description
docs/docs/usage/notifications.md Rewritten guide for configuring triggers/digests, resource/event types, filters, notifiers, and the in-app notification center.
docs/docs/usage/evaluate/findings/findings.md Clarifies asset scope (all inventory categories except security platforms) and updates wording to “Impacted assets”.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +49 to +50
| Finding | Asset filters cover every asset category except security platforms |
| Asset | Hosts, cloud resources, web applications, AI targets, and the other inventory categories |
| Creation | A new entity of the selected type is created |
| Modification | An existing entity is updated |
| Deletion | An entity is deleted |
| Score degradation | Only for **Scenario**. A Simulation score drops below the previous run of the same Scenario (the former notification-rule trigger) |
@filigran-cla-bot filigran-cla-bot Bot removed the cla:pending CLA signature required. label Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Contribution from the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs(usage): document notification triggers and asset-scoped filter options

3 participants