New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 791220 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

Invalid Date on the first column

Project Member Reported by st...@chromium.org, Dec 2 2017

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/f8845fcf46380fabd49b63a849f6524d3734ca25

commit f8845fcf46380fabd49b63a849f6524d3734ca25
Author: Brandon Wylie <wylieb@chromium.org>
Date: Mon Dec 04 19:10:14 2017

Revert "[Findit] Flake Analyzer - Convert timestamp to local time."

This reverts commit 2daaf183b9c9924364625ba7b7920bdf97776ead.

Reason for revert: Doesn't work as intended.

Original change's description:
> [Findit] Flake Analyzer - Convert timestamp to local time.
>
> This will allow the user to make sense of the date without having to
> convert from UTC.
>
> Bug:786571
> Change-Id: I6b6c0a9db27ee15a9d1b2ef7601775db4dc6ccb7
> Reviewed-on: https://chromium-review.googlesource.com/795112
> Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>
> Reviewed-by: Shuotao Gao <stgao@chromium.org>
> Commit-Queue: Brandon Wylie <wylieb@chromium.org>

TBR=stgao@chromium.org,robertocn@chromium.org,lijeffrey@chromium.org,chanli@chromium.org,wylieb@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  791220 
Change-Id: Ic55865ac35a88ad9fc1ef6a552c248d78b80c006
Reviewed-on: https://chromium-review.googlesource.com/806639
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Reviewed-by: Brandon Wylie <wylieb@chromium.org>

[modify] https://crrev.com/f8845fcf46380fabd49b63a849f6524d3734ca25/appengine/findit/templates/flake/dashboard.html
[modify] https://crrev.com/f8845fcf46380fabd49b63a849f6524d3734ca25/appengine/findit/handlers/flake/list_flakes.py
[modify] https://crrev.com/f8845fcf46380fabd49b63a849f6524d3734ca25/appengine/findit/handlers/flake/test/list_flakes_test.py

Comment 2 by wylieb@chromium.org, Jan 17 2018

Status: Fixed (was: Assigned)

Sign in to add a comment