Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MERGE WITH #39] Fixed filters in dashboard #37

Merged
merged 3 commits into from
May 3, 2016
Merged

[MERGE WITH #39] Fixed filters in dashboard #37

merged 3 commits into from
May 3, 2016

Conversation

acarb95
Copy link

@acarb95 acarb95 commented Apr 11, 2016

Changed File Reconstruction Dashboard and SMTP Trends dashboard to use filters instead of a specific search in the visualization. Ran on VM (didn't have SMTP or File Recon traffic) and the filters were there.

Please pull down and double check that it works on a system with SMTP and File Recon traffic.

Also fixed Top 10 Attachment Types (bar graph), it was displaying more than the top 10.

Merge after Craig's PR: #39

@alexweltman
Copy link

@acarb95 why all the file renames/version changes? is that how @craig-cogdill said to do versioning?

@@ -2,7 +2,7 @@
"title": "Top 10 Email Sender Domains",
"visState": "{\"type\":\"pie\",\"params\":{\"shareYAxis\":true,\"addTooltip\":true,\"addLegend\":true,\"isDonut\":false},\"aggs\":[{\"id\":\"1\",\"type\":\"count\",\"schema\":\"metric\",\"params\":{}},{\"id\":\"2\",\"type\":\"terms\",\"schema\":\"segment\",\"params\":{\"field\":\"SenderDomain.raw\",\"size\":10,\"order\":\"desc\",\"orderBy\":\"1\"}}],\"listeners\":{}}",
"description": "",
"version": 1,
"version": 2,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't need to update this version since this visualization hasn't changed

@craig-cogdill
Copy link

This should actually get merged AFTER my PR in Kibana for redo-ing the python scripts for inserting everything. It'll allow us to test more accurately with all the version changes.

@acarb95
Copy link
Author

acarb95 commented Apr 29, 2016

Okay! I'll add that to the title.

@acarb95 acarb95 changed the title Fixed filters in dashboard [DO NOT MERGE] Fixed filters in dashboard Apr 29, 2016
@craig-cogdill
Copy link

Looks like it works great on 190. Doesn't necessarily have to be merged AFTER the other one, but they should be merged on the same day so that they're both in a build

@craig-cogdill
Copy link

👍

@acarb95 acarb95 changed the title [DO NOT MERGE] Fixed filters in dashboard [MERGE WITH #39] Fixed filters in dashboard Apr 29, 2016
@craig-cogdill craig-cogdill merged commit 46b91b5 into logrhythm:4.1.4_NM May 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants