getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.16k stars 4.2k forks source link

Alert preview IF condition behavior #69774

Open kerenkhatiwada opened 6 months ago

kerenkhatiwada commented 6 months ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

Reported by a user: "I'm using the default of "IF any of these filters match" without defining any filters. However, that causes the Preview section to always show "We couldn't find any issues that would've triggered your rule"

Instead if there's no additional filters specified then I wouldn't expect the "any/all/none" setting for filters to matter (and also if there aren't any filters I'd definitely expect "any" to match if "all" matches)."

Reproduced below:

https://github.com/getsentry/sentry/assets/99350819/daab1313-25d2-436d-ad40-b79dd73472b8

Expected Result

if there's no additional filters specified in the IF condition of an alert, any/all/none would return the same preview results

Actual Result

Currently, only all and none return results in the preview

Product Area

Alerts

Link

https://org2-w7.sentry.io/alerts/rules/react/15146378/details/

DSN

No response

Version

No response

getsantry[bot] commented 6 months ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 6 months ago

Routing to @getsentry/product-owners-alerts for triage ⏲️