[GitHub] [metron] mmiklavc commented on issue #1443: METRON-2150: [UI] User not able to filter by multiple values of the same field on Alerts UI

2019-07-02 Thread GitBox
mmiklavc commented on issue #1443: METRON-2150: [UI] User not able to filter by 
multiple values of the same field on Alerts UI
URL: https://github.com/apache/metron/pull/1443#issuecomment-507778601
 
 
   +1 by inspection, thanks @tiborm. Side note, thanks for taking the 
initiative to make minor improvements along the way, e.g. extracting strings 
like `guid` into constants. Good stuff!


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [metron] mmiklavc commented on issue #1443: METRON-2150: [UI] User not able to filter by multiple values of the same field on Alerts UI

2019-06-28 Thread GitBox
mmiklavc commented on issue #1443: METRON-2150: [UI] User not able to filter by 
multiple values of the same field on Alerts UI
URL: https://github.com/apache/metron/pull/1443#issuecomment-506806943
 
 
   @anandsubbu is 2167 an issue introduced by this PR, or a pre-existing bug?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services