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

make all timestamps the same format #103

Closed
edaniszewski opened this issue Mar 23, 2018 · 3 comments
Closed

make all timestamps the same format #103

edaniszewski opened this issue Mar 23, 2018 · 3 comments
Assignees
Labels

Comments

@edaniszewski
Copy link
Contributor

Description

not all timestamps are returned in the same format.

test

{
  "status":"ok",
  "timestamp":"2018-03-23 14:06:40.147079"
}

read

{
  "type":"temperature",
  "data":{
    "temperature":{
      "value":81.0,
      "timestamp":"2018-03-23T14:05:31.6192819Z",
      "unit":{
        "symbol":"C",
        "name":"degrees celsius"
      }
    }
  }
}
@edaniszewski edaniszewski self-assigned this Mar 23, 2018
@edaniszewski
Copy link
Contributor Author

@MatthewHink
Copy link
Contributor

Side thing. Are all timestamps UTC? Sites will be in different places and could be distributed at some point.

@edaniszewski
Copy link
Contributor Author

Yes - for now all timestamps are UTC. I've looked into timezoning on timestamps, but it doesn't seem all that straightforward, so I just left it as UTC for now. We can definitely work towards getting proper time zone support in the future.

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

No branches or pull requests

2 participants