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

Global timezones #5134

Merged
merged 2 commits into from
Oct 20, 2015
Merged

Global timezones #5134

merged 2 commits into from
Oct 20, 2015

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Oct 16, 2015

Enables changing the application-wide timezone. Closes #1600

image

@@ -20,6 +23,12 @@ define(function () {
value: 'MMMM Do YYYY, HH:mm:ss.SSS',
description: 'When displaying a pretty formatted date, use this format',
},
'dateFormat:tz': {
value: 'Default',
Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe this should be Auto detect or something similar?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd use either "browser" or "local" to try to make it clear how the detection is being done. Old versions of kibana used "browser"

@rashidkpc
Copy link
Contributor

Few small comments, otherwise looks good.

@rashidkpc rashidkpc assigned jbudz and unassigned rashidkpc Oct 20, 2015
@jbudz jbudz assigned w33ble and unassigned jbudz Oct 20, 2015
@w33ble
Copy link
Contributor

w33ble commented Oct 20, 2015

Spent some quality time with this one. Changing the time zone does in fact change the time_zone in the date histogram request, and the resulting data does is correct. LGTM!

w33ble added a commit that referenced this pull request Oct 20, 2015
@w33ble w33ble merged commit fce1637 into elastic:master Oct 20, 2015
@ahartimvu
Copy link

way to go!

@tbragin tbragin mentioned this pull request Nov 3, 2015
9 tasks
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.

5 participants