feat(alerting): make it possible to pause from alert tab

pkg/api/alerting.go

@@ -252,13 +252,17 @@ func NotificationTest(c *middleware.Context, dto dtos.NotificationTestCommand) R
 	return ApiSuccess("Test notification sent")
 	return ApiSuccess("Test notification sent")
 }
 }
 
 
-//POST /api/alerts/:alertId/pause
-func PauseAlert(c *middleware.Context, cmd models.PauseAlertCommand) Response {
-	cmd.OrgId = c.OrgId
-	cmd.AlertId = c.ParamsInt64(":alertId")
+//POST /api/pause-alert
+func PauseAlert(c *middleware.Context, dto dtos.PauseAlertCommand) Response {
+	alertId, err := getAlertIdForRequest(c.OrgId, dto.AlertId, dto.PanelId, dto.DashboardId)
+	if err != nil {
+		return ApiError(400, "Bad request", err)
+	}
 
 
-	if cmd.AlertId == 0 {
-		return ApiError(400, "Missing alert id", nil)
+	cmd := models.PauseAlertCommand{
+		OrgId:   c.OrgId,
+		AlertId: alertId,
+		Paused:  dto.Paused,
 	}
 	}
 
 
 	if err := bus.Dispatch(&cmd); err != nil {
 	if err := bus.Dispatch(&cmd); err != nil {
@@ -278,11 +282,7 @@ func PauseAlert(c *middleware.Context, cmd models.PauseAlertCommand) Response {
 	return Json(200, result)
 	return Json(200, result)
 }
 }
 
 
-func getAlertIdForRequest(c *middleware.Context) (int64, error) {
-	alertId := c.QueryInt64("alertId")
-	panelId := c.QueryInt64("panelId")
-	dashboardId := c.QueryInt64("dashboardId")
-
+func getAlertIdForRequest(orgId, alertId, panelId, dashboardId int64) (int64, error) {
 	if alertId == 0 && dashboardId == 0 && panelId == 0 {
 	if alertId == 0 && dashboardId == 0 && panelId == 0 {
 		return 0, fmt.Errorf("Missing alertId or dashboardId and panelId")
 		return 0, fmt.Errorf("Missing alertId or dashboardId and panelId")
 	}
 	}
@@ -290,7 +290,7 @@ func getAlertIdForRequest(c *middleware.Context) (int64, error) {
 	if alertId == 0 {
 	if alertId == 0 {
 		//fetch alertId
 		//fetch alertId
 		query := models.GetAlertsQuery{
 		query := models.GetAlertsQuery{
-			OrgId:       c.OrgId,
+			OrgId:       orgId,
 			DashboardId: dashboardId,
 			DashboardId: dashboardId,
 			PanelId:     panelId,
 			PanelId:     panelId,
 		}
 		}

pkg/api/api.go

@@ -252,12 +252,14 @@ func Register(r *macaron.Macaron) {
 
 
 		r.Group("/alerts", func() {
 		r.Group("/alerts", func() {
 			r.Post("/test", bind(dtos.AlertTestCommand{}), wrap(AlertTest))
 			r.Post("/test", bind(dtos.AlertTestCommand{}), wrap(AlertTest))
-			r.Post("/:alertId/pause", ValidateOrgAlert, bind(m.PauseAlertCommand{}), wrap(PauseAlert))
+
 			r.Get("/:alertId", ValidateOrgAlert, wrap(GetAlert))
 			r.Get("/:alertId", ValidateOrgAlert, wrap(GetAlert))
 			r.Get("/", wrap(GetAlerts))
 			r.Get("/", wrap(GetAlerts))
 			r.Get("/states-for-dashboard", wrap(GetAlertStatesForDashboard))
 			r.Get("/states-for-dashboard", wrap(GetAlertStatesForDashboard))
 		})
 		})
 
 
+		r.Post("/pause-alert", bind(dtos.PauseAlertCommand{}), wrap(PauseAlert))
+
 		r.Get("/alert-notifications", wrap(GetAlertNotifications))
 		r.Get("/alert-notifications", wrap(GetAlertNotifications))
 
 
 		r.Group("/alert-notifications", func() {
 		r.Group("/alert-notifications", func() {

pkg/api/dtos/alerting.go

@@ -63,4 +63,5 @@ type PauseAlertCommand struct {
 	AlertId     int64 `json:"alertId"`
 	AlertId     int64 `json:"alertId"`
 	DashboardId int64 `json:"dashboardId"`
 	DashboardId int64 `json:"dashboardId"`
 	PanelId     int64 `json:"panelId"`
 	PanelId     int64 `json:"panelId"`
+	Paused      bool  `json:"paused"`
 }
 }

pkg/models/alert.go

@@ -103,8 +103,8 @@ type SaveAlertsCommand struct {
 
 
 type PauseAlertCommand struct {
 type PauseAlertCommand struct {
 	OrgId   int64
 	OrgId   int64
-	AlertId int64 `json:"alertId"`
-	Paused  bool  `json:"paused"`
+	AlertId int64
+	Paused  bool
 }
 }
 
 
 type SetAlertStateCommand struct {
 type SetAlertStateCommand struct {

public/app/features/alerting/alert_list_ctrl.ts

@@ -47,10 +47,11 @@ export class AlertListCtrl {
     var alert = _.find(this.alerts, {id: alertId});
     var alert = _.find(this.alerts, {id: alertId});
 
 
     var payload = {
     var payload = {
-      paused: alert.state !== "paused"
+      paused: alert.state !== "paused",
+      alertId: alert.id
     };
     };
 
 
-    this.backendSrv.post(`/api/alerts/${alertId}/pause`, payload).then(result => {
+    this.backendSrv.post(`/api/pause-alert`, payload).then(result => {
       alert.state = result.state;
       alert.state = result.state;
       alert.stateModel = alertDef.getStateDisplayModel(result.state);
       alert.stateModel = alertDef.getStateDisplayModel(result.state);
     });
     });