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

Fix timezone handling #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix timezone handling #46

wants to merge 1 commit into from

Conversation

mattbk
Copy link
Owner

@mattbk mattbk commented Nov 20, 2018

Datetimes were being stored as local rather than UTC. I have fixed this so far, which means in the local db I am using there is a big jump between 2018-11-19 20:46:02 and 2018-11-20 02:47:02, where the former is in America/Chicago and the latter is in UTC.

Also fixed so far is the graph showing the wrong datetimes on the x-axis even though they are correct in the range picker.

  • set default timezone somewhere. Browser timestamp isn't set until a radio button is pressed, which is not good. See images below.
  • (local) fix earlier db entries

screen_shot_2018-11-19_at_8_59_39_pm-2

screen_shot_2018-11-19_at_8_59_57_pm

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.

1 participant