Closed
Bug 1442003
Opened 8 years ago
Closed 8 years ago
Lando API should return all dates as iso 8601 with timezones
Categories
(Conduit :: Lando, enhancement)
Conduit
Lando
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: smacleod, Assigned: smacleod)
References
Details
(Keywords: conduit-triaged)
Attachments
(2 files)
Some of the dates lando API is returning are in iso 8601, but missing the timezone component. While these dates are UTC as intended, the spec states that timestamps without the timezone should be considered local time. We should make sure all the returned timestamps have the timezone (UTC).
Examples:
- \landings:
- 'created_at': '2018-02-09T20:48:20.293139'
- 'updated_at': '2018-02-09T20:48:22.086858'
Updated•8 years ago
|
Keywords: conduit-triaged
Comment 1•8 years ago
|
||
I think this bug is important enough to do now because it is actually causing the UI to lie to the user as per the screenshot I've attached. The diff time is correct because it properly has the UTC timezone set by lando-api (fixed in https://bugzilla.mozilla.org/show_bug.cgi?id=1441607).
Comment 2•8 years ago
|
||
Comment 4•8 years ago
|
||
I think this can be filed (and communicated) as a known bug, since it's straightforward enough (mis)behaviour.
Flags: needinfo?(mcote)
Comment 5•8 years ago
|
||
Comment 6•8 years ago
|
||
Comment on attachment 8960336 [details]
landings: fix Landing dates.
Byron Jones ‹:glob› has approved the revision.
https://phabricator.services.mozilla.com/D774
Attachment #8960336 -
Flags: review+
| Assignee | ||
Updated•8 years ago
|
Assignee: nobody → smacleod
| Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•