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

Feature sharelink #1155

Closed
wants to merge 14 commits into from
Closed

Feature sharelink #1155

wants to merge 14 commits into from

Conversation

awheeler
Copy link

This implements the feature request #717.
Additionally, it adds consistent URL arg support to all dashboards.

This is a re-pull of pull #1100, with a branch rename to facilitate managing multiple pull requests.

module.filter('shareLink', function($location) {
return function(event) {
var from = new Date(Date.parse(event._source[this.panel.timeField])-60000);
var to = new Date(Date.parse(event._source[this.panel.timeField])+60000);
Copy link
Contributor

Choose a reason for hiding this comment

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

Date.parse is generally unreliable for parsing timestamps, this will fail for any data that doesn't use iso8601.

@rashidkpc
Copy link
Contributor

This is really very logstash specific and makes a number of assumptions. I understand the benefit of trying to link to the event on the same dashboard, but given the compromises that must be made I really think one of the options we discussed before is a better solution.

… feature_sharelink

Conflicts:
	src/app/panels/table/module.html
@rashidkpc rashidkpc removed the wontfix label Oct 6, 2014
@w33ble w33ble closed this Oct 6, 2014
@spalger
Copy link
Contributor

spalger commented Oct 6, 2014

Woops! Sorry about that! We recently replaced the master branch with Kibana 4. This action force closed all of the old pull requests against master. We will be reviewing these on a case-by-case basis and creating new tickets as necessary.

The good news is that many long requested features can be found in Kibana 4, and we're being entirely open about our roadmap. Check out the roadmap tickets (which we're still filling in) here.

If you're looking for the old Kibana 3 code you can find it here.

@w33ble
Copy link
Contributor

w33ble commented Oct 8, 2014

We're stopping development on Kibana 3 and focusing on Kibana 4 going forward. This can't be merged as-is, but is tagged for completion in Kibana 4 Beta 3 - you can track progress in #717

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants