You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
sushovan majumdar 11 Jun at 18:24 @satterly We have use case where we are managing a lot alerts and sometimes support team is creating blackout rules with just environment:production which ends up blocking all alerts. They do create that massive blackout rule by mistake which blocks all alerts. Would you consider adding a validation check in blackout form or warning message to users that by doing it they will be effectively stopping all alerts and have them confirm that ?
+1
grinning
tada
4 replies
Nick 5 months ago
Yes I could add a confirm dialog if it's for the entire environment.
Cody Stevens 5 months ago
you might consider doing that for Origin as well
👍
1
sushovan majumdar 5 months ago
yes that confirm dialog will help the ops/support work. Thank you
sushovan majumdar 4 months ago
If you want I can create an issue for it so we can track it? otherwise I keep on asking here 🙂
The text was updated successfully, but these errors were encountered:
sushovan majumdar 11 Jun at 18:24
@satterly We have use case where we are managing a lot alerts and sometimes support team is creating blackout rules with just environment:production which ends up blocking all alerts. They do create that massive blackout rule by mistake which blocks all alerts. Would you consider adding a validation check in blackout form or warning message to users that by doing it they will be effectively stopping all alerts and have them confirm that ?
+1
grinning
tada
4 replies
Nick 5 months ago
Yes I could add a confirm dialog if it's for the entire environment.
Cody Stevens 5 months ago
you might consider doing that for Origin as well
👍
1
sushovan majumdar 5 months ago
yes that confirm dialog will help the ops/support work. Thank you
sushovan majumdar 4 months ago
If you want I can create an issue for it so we can track it? otherwise I keep on asking here 🙂
The text was updated successfully, but these errors were encountered: