Follow-up from "Add search argument for AlertStatusCountsResolver"
The following discussions from !34596 (merged) should be addressed:
-
@splattael started a discussion: Non-blocking: While reviewing the specs below I wasn't sure what
alert_1
andalert_2
means so I had to look it up.What do you think of renaming this
let
to e.g.alert_resolved
(orresolved_alert
)? Similar foralert_2
, wouldalert_triggered
work? -
@splattael started a discussion: Non-blocking: We could clarify the name of the
let
s and rename them toalert_resolved
,alert_ignored
, andalert_triggered
.WDYT?