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

Add timezone to commons 'application' schemas #22

Closed
blootsvoets opened this issue Apr 11, 2017 · 1 comment
Closed

Add timezone to commons 'application' schemas #22

blootsvoets opened this issue Apr 11, 2017 · 1 comment

Comments

@blootsvoets
Copy link
Member

Right now, we send time values in seconds in UTC, however, for a proper day analysis, we probably also need to send timezone somewhere. This doesn't necessarily have to be sent in each message, but it should at least be sent by the application. The most direct would be to send the UTC offset, where an offset of one hour compared to UTC would be "+01:00", "+0100", or simply "+01". Using a time zone string would reveal more about someones location.

@blootsvoets blootsvoets changed the title Add timezone to commons application Add timezone to commons 'application' schemas Apr 11, 2017
@yatharthranjan
Copy link
Member

Close as fixed

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

No branches or pull requests

2 participants