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

Timezone correction across panels #341

Closed
wants to merge 5 commits into from
Closed

Conversation

dhawani
Copy link

@dhawani dhawani commented Aug 8, 2013

This pull will help fix #95, #69, #70, #337, #340

  • Timezone has been introduced in dashboard configuration. As this is the common setting to be used across panels viz. table, histogram and timepicker.
  • In table configuration, added support to define timeformat
  • From histogram removed timezone setting and pointed to use the dashboard one.
  • In config.js, zone files can be specified which needs to be loaded.

Timezone has been introduced in dashboard configuration. As this is the common setting to be used across panels viz. table, histogram and timepicker.
In table configuration, added support to define timeformat
From histogram removed timezone setting and pointed to use the dashboard one.
In config.js, specify timezone directory according to server (and hence modified settings.js) Not required zones can be removed.
@willejs
Copy link

willejs commented Aug 18, 2013

Really keen to see this merged!

@frankyaorenjie
Copy link

Looking forward to this merge

@dhawani
Copy link
Author

dhawani commented Aug 21, 2013

Closing this for now.

@dhawani dhawani closed this Aug 21, 2013
@willejs
Copy link

willejs commented Aug 21, 2013

why has this been closed instead of merged? we still lack utc time zone for time picker in the latest commit.

@dhawani
Copy link
Author

dhawani commented Aug 21, 2013

@rashidkpc is working on. He is implementing server side solution.

@Gnarfoz
Copy link

Gnarfoz commented Oct 30, 2013

What does "server side" even mean for a html5+js web application that has no backend?
Is it going to be implemented in ElasticSearch instead?

@rashidkpc
Copy link
Contributor

@Gnarfoz thats the idea, yes.

@Gnarfoz
Copy link

Gnarfoz commented Oct 30, 2013

Nice, I'll keep an eye on ElasticSearch changes, then. :-)

@iamcrunch
Copy link

Is there an issue number @rashidkpc is using to track this work? I'd like to keep up on progress.

@blufor
Copy link

blufor commented Nov 19, 2013

@rashidkpc +1 for the link of the issue in elasticsearch project (already tried searching for it)

spalger pushed a commit to spalger/kibana that referenced this pull request Mar 3, 2016
harper-carroll pushed a commit to harper-carroll/kibana that referenced this pull request Jul 11, 2016
w33ble added a commit to w33ble/kibana that referenced this pull request Sep 13, 2018
* fix: remove debugging output

* feat: axisConfig function

* feat: add axis config ui

* feat: use axis config in plot chart

* feat: add horizontalBars arg to seriesStyle

use the setting in the plot function
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.

@timestamp in local time
8 participants