Merge pull request #505 from EverythingMe/small_fixes

Fix: Update URL after creating an alert
getredash · Jul 22, 2015 · da56dc8 · da56dc8
2 parents bff4d31 + 02582ca
commit da56dc8
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/rd_ui/app/scripts/controllers/alerts.js b/rd_ui/app/scripts/controllers/alerts.js
@@ -46,7 +46,7 @@
     ];
   };
 
-  var AlertCtrl = function($scope, $routeParams, growl, Query, Events, Alert) {
+  var AlertCtrl = function($scope, $routeParams, $location, growl, Query, Events, Alert) {
     $scope.$parent.pageTitle = "Alerts";
 
     $scope.alertId = $routeParams.alertId;
@@ -97,8 +97,11 @@
         $scope.alert.name = $scope.getDefaultName();
       }
 
-      $scope.alert.$save(function() {
+      $scope.alert.$save(function(alert) {
         growl.addSuccessMessage("Saved.");
+        if ($scope.alertId === "new") {
+           $location.path('/alerts/' + alert.id).replace();
+        }
       }, function() {
         growl.addErrorMessage("Failed saving alert.");
       });
@@ -166,6 +169,6 @@
 
   angular.module('redash.controllers')
     .controller('AlertsCtrl', ['$scope', 'Events', 'Alert', AlertsCtrl])
-    .controller('AlertCtrl', ['$scope', '$routeParams', 'growl', 'Query', 'Events', 'Alert', AlertCtrl])
+    .controller('AlertCtrl', ['$scope', '$routeParams', '$location', 'growl', 'Query', 'Events', 'Alert', AlertCtrl])
 
 })();