getsentry / getsentry/sentry

Extend conflict detection to alert filters

Open
#73,999 3 comments 0 reactions 0 assignees View on GitHub
Feature Product Area: Alerts
Dominant language
Python
Stars
44.8k
Forks
4.9k
Avg merge
22h 21m
Merged PRs (30d)
586

Description

### Problem Statement

Currently sentry alerts the user when two mutual exclusive conditions to trigger an alert are selected. This detection are not applied to the alerts filters.

### Solution Brainstorm

Add conflicts warnings to filters as well, the example below has two conflicting rule pairs but not error message is shown as in the conditions.

![Screenshot 2024-07-09 at 10 57 49](https://github.com/getsentry/sentry/assets/64906376/ef036cce-fd9d-442e-bbfe-ea7c08e845a4)

### Product Area

Alerts

Contributor guide

Open the contributing guide

Research direction

Start by tracing the existing conflict detection for alert conditions, then compare how alert filters are represented and validated. Done means the same mutual-exclusion warnings appear for conflicting filter rule pairs, including the example shown.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.