Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(146)

Issue 2706813003: Add new endpoint to get bug details as JSON. (Closed)

Created:
3 years, 10 months ago by sullivan
Modified:
3 years, 10 months ago
Reviewers:
eakuefner, shatch
CC:
catapult-reviews_chromium.org, perf-dashboard-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Added basic support for bisects. #

Total comments: 2

Patch Set 3 : Added polymer ui #

Total comments: 1

Patch Set 4 : Add oauth2 #

Patch Set 5 : fixed error messaging when bugs fail to load #

Patch Set 6 : Fix owner #

Patch Set 7 : Actually link bugs #

Total comments: 1

Patch Set 8 : addressed review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+508 lines, -5 lines) Patch
M dashboard/dashboard/benchmark_health_report.py View 1 2 3 4 5 6 7 2 chunks +2 lines, -0 lines 0 comments Download
A dashboard/dashboard/bug_details.py View 1 2 3 4 5 6 7 1 chunk +81 lines, -0 lines 0 comments Download
A dashboard/dashboard/bug_details_test.py View 1 1 chunk +108 lines, -0 lines 0 comments Download
M dashboard/dashboard/dispatcher.py View 2 chunks +2 lines, -0 lines 0 comments Download
M dashboard/dashboard/elements/benchmark-health-report-details.html View 1 2 4 chunks +17 lines, -0 lines 0 comments Download
M dashboard/dashboard/elements/benchmark-health-report-page-test.html View 1 2 3 chunks +52 lines, -5 lines 0 comments Download
A dashboard/dashboard/elements/bug-details.html View 1 2 3 4 5 6 1 chunk +154 lines, -0 lines 0 comments Download
A dashboard/dashboard/elements/bug-details-test.html View 1 2 3 4 1 chunk +72 lines, -0 lines 0 comments Download
M dashboard/dashboard/services/issue_tracker_service.py View 1 2 3 4 5 6 7 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (9 generated)
sullivan
The plan is: * This CL adds an endpoint that gets bug data. I'll likely ...
3 years, 10 months ago (2017-02-20 22:10:13 UTC) #2
sullivan
I added some basic data about bisects. https://codereview.chromium.org/2706813003/diff/20001/dashboard/dashboard/bug_details.py File dashboard/dashboard/bug_details.py (right): https://codereview.chromium.org/2706813003/diff/20001/dashboard/dashboard/bug_details.py#newcode52 dashboard/dashboard/bug_details.py:52: 'owner': bug_details['owner']['name'], ...
3 years, 10 months ago (2017-02-20 22:48:18 UTC) #3
sullivan
Demo: https://dev-sullivan-b2373ed2-dot-chromeperf.appspot.com/benchmark_health_report?benchmark=speedometer The 500 errors on bugs are due to restrict-view-google. Need to clean that ...
3 years, 10 months ago (2017-02-21 03:25:35 UTC) #4
sullivan
On 2017/02/21 03:25:35, sullivan wrote: > Demo: > https://dev-sullivan-b2373ed2-dot-chromeperf.appspot.com/benchmark_health_report?benchmark=speedometer > > The 500 errors on ...
3 years, 10 months ago (2017-02-23 15:55:04 UTC) #9
shatch
lgtm Curious what's up with those 500 errors but don't need to block on it ...
3 years, 10 months ago (2017-02-23 17:27:22 UTC) #10
sullivan
On 2017/02/23 17:27:22, shatch wrote: > lgtm > > Curious what's up with those 500 ...
3 years, 10 months ago (2017-02-23 22:27:27 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2706813003/140001
3 years, 10 months ago (2017-02-23 22:30:24 UTC) #14
commit-bot: I haz the power
3 years, 10 months ago (2017-02-23 23:28:30 UTC) #17
Message was sent while issue was closed.
Committed patchset #8 (id:140001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Powered by Google App Engine
This is Rietveld 408576698