-
Notifications
You must be signed in to change notification settings - Fork 18
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
Added field name capitalization to discover and visualize #51
Added field name capitalization to discover and visualize #51
Conversation
alexweltman
commented
Jul 5, 2016
- Added field name capitalization to queries on the discover and visualize tabs
- fixed broken time ranges being sent to audit log
var notify = new Notifier({ | ||
location: 'Dashboard' | ||
}); | ||
var notify = new Notifier({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here I just removed the link function cause it's no longer needed, that functionality was moved to the searchAuditor service and to $scope.filterResults. The rest of the changes is fixing indentations
}; | ||
var _ = require('lodash'); | ||
var moment = require('moment'); | ||
var angular = require('angular'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we requiring angular?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kibana does this on every file that uses an angular method like angular.copy
ha ha ha ... Is that a plus one @sammccord ? |
@alexweltman is this for 3.1.2? |
👍 reviewed & tested |