Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Commit

Permalink
HAWKULAR-690 Fix alerts message on cancelling modal
Browse files Browse the repository at this point in the history
  • Loading branch information
lucasponce committed Oct 13, 2015
1 parent 2e035c8 commit 15c8997
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,6 @@ module HawkularMetrics {

modalInstance.result.then(angular.noop, () => {
log.info('Jvm Alert Setup modal dismissed at: ' + new Date());
this.NotificationsService.alertSettingsSaved();
});
}, () => {
this.$log.error('Missing and unable to create new JVM Alert triggers.');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ module HawkularMetrics {
constructor(private $scope:any,
private HawkularAlertsManager:IHawkularAlertsManager,
private ErrorsManager:IErrorsManager,
private NotificaitonsService:INotificationsService,
private NotificationsService:INotificationsService,
private $log:ng.ILogService,
private $q:ng.IQService,
private $rootScope:IHawkularRootScope,
Expand Down Expand Up @@ -246,7 +246,6 @@ module HawkularMetrics {
});

modalInstance.result.then(angular.noop, () => {
this.NotificaitonsService.alertSettingsSaved();
log.debug('Web Alert Setup modal dismissed at: ' + new Date());
});
}, () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,6 @@ module HawkularMetrics {
modalInstance.result.then(function (selectedItem) {
this.selected = selectedItem;
}, () => {
this.NotificationsService.alertSettingsSaved();
logger.info('Modal dismissed at: ' + new Date());
});
}
Expand Down

0 comments on commit 15c8997

Please sign in to comment.