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

Include timezone offset in machine-readable timestamp #7982

Closed
aaronpk opened this issue Jul 8, 2018 · 2 comments
Closed

Include timezone offset in machine-readable timestamp #7982

aaronpk opened this issue Jul 8, 2018 · 2 comments

Comments

@aaronpk
Copy link

aaronpk commented Jul 8, 2018

Toots are displayed in (someone's) local time, but the machine readable date is always rendered in UTC.

The machine readable date should be rendered as an ISO8601 timestamp with the same timezone offset that the human readable date is rendered as. That applies both here and here.

@nightpool
Copy link
Member

nightpool commented Jul 8, 2018 via email

@aaronpk
Copy link
Author

aaronpk commented Jul 8, 2018

I didn't realize that was changing the displayed time client-side.

Using Z for the offset is the lazy way to avoid the problem. As long as the timezone offset is included in the timestamp, it still refers to an absolute point in time and can be easily converted to other timezones. (e.g. 2018-07-08T06:57:00-0700)

Ideally the timestamps should be rendered in the timezone the post was authored in, as that can provide additional context for a post (did they post this in their morning or late at night?) But, if you don't have that information available then there isn't much you can do.

@aaronpk aaronpk closed this as completed Jul 8, 2018
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

No branches or pull requests

2 participants