[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-15 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/1266 METRON-1875: Expose configurable global settings in the Alerts UI ## Contributor Comments This PR exposes a JSON file that can be used to configure the Alerts UI. Properties in this file are

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234283406 --- Diff: metron-interface/metron-alerts/src/app/service/app-config.service.ts --- @@ -0,0 +1,40 @@ +/** + * Licensed to the Apache Software Foundat

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234287609 --- Diff: metron-interface/metron-alerts/src/app/alerts/alerts-list/table-view/table-view.component.spec.ts --- @@ -31,12 +31,20 @@ import { UpdateService

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234287807 --- Diff: metron-interface/metron-alerts/src/app/alerts/alerts-list/tree-view/tree-view.component.spec.ts --- @@ -32,6 +32,14 @@ import { UpdateService } f

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234287914 --- Diff: metron-interface/metron-alerts/src/app/app.module.ts --- @@ -48,13 +48,11 @@ import { GlobalConfigService } from './service/global-config.service'

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288056 --- Diff: metron-interface/metron-alerts/src/app/pcap/service/pcap.service.spec.ts --- @@ -33,6 +33,7 @@ import { PcapStatusResponse } from '../model/pcap-

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288083 --- Diff: metron-interface/metron-alerts/src/app/pcap/service/pcap.service.ts --- @@ -25,6 +25,7 @@ import { HttpUtil } from '../../utils/httpUtil'; im

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288118 --- Diff: metron-interface/metron-alerts/src/app/service/alerts.service.ts --- @@ -22,14 +22,15 @@ import {Alert} from '../model/alert'; import { HttpC

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288233 --- Diff: metron-interface/metron-alerts/src/app/service/global-config.service.ts --- @@ -21,14 +21,15 @@ import { Injectable } from '@angular/core'; i

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288029 --- Diff: metron-interface/metron-alerts/src/app/pcap/pcap-panel/pcap-panel.component.spec.ts --- @@ -28,6 +28,7 @@ import { PcapRequest } from '../model/p

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288184 --- Diff: metron-interface/metron-alerts/src/app/service/authentication.service.ts --- @@ -22,26 +22,28 @@ import {Router} from '@angular/router'; impo

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288308 --- Diff: metron-interface/metron-alerts/src/app/service/search.service.ts --- @@ -30,6 +30,7 @@ import { RestError } from '../model/rest-error'; impor

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288335 --- Diff: metron-interface/metron-alerts/src/app/service/update.service.ts --- @@ -28,6 +28,7 @@ import {Utils} from '../utils/utils'; import {Patch} f

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234288280 --- Diff: metron-interface/metron-alerts/src/app/service/meta-alert.service.ts --- @@ -26,14 +26,15 @@ import {Alert} from '../model/alert'; import { H

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234289363 --- Diff: metron-interface/metron-alerts/src/app/pcap/service/pcap.service.ts --- @@ -47,47 +48,47 @@ export class PcapService { public submitRequest

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234293206 --- Diff: metron-interface/metron-alerts/src/app/service/app-config.service.ts --- @@ -0,0 +1,40 @@ +/** + * Licensed to the Apache Software Found

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1266#discussion_r234296496 --- Diff: metron-interface/metron-alerts/src/app/service/app-config.service.ts --- @@ -0,0 +1,40 @@ +/** + * Licensed to the Apache Software Foundat

[GitHub] metron pull request #1266: METRON-1875: Expose configurable global settings ...

2018-11-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1266 ---