Github user tiborm commented on a diff in the pull request:
https://github.com/apache/metron/pull/1240#discussion_r226589952
--- Diff:
metron-interface/metron-alerts/src/app/alerts/alerts-list/table-view/table-view.component.ts
---
@@ -259,19 +259,25 @@ export class TableViewComponent implements OnInit,
OnChanges, OnDestroy {
}
deleteOneAlertFromMetaAlert($event, alert: Alert, metaAlertIndex:
number) {
- this.metronDialogBox.showConfirmationMessage('Do you wish to remove
the alert from the meta alert?').subscribe(response => {
- if (response) {
+ let confirmedSubscription = this.dialogService.confirm('Do you wish to
remove the alert from the meta alert?').subscribe(r => {
--- End diff --
I would use const here (and all the other places) too.
---
