Merge lp://staging/~stevanr/linaro-ci-dashboard/build-detail-view into lp://staging/linaro-ci-dashboard

Proposed by Stevan Radaković
Status: Merged
Approved by: Milo Casagrande
Approved revision: 49
Merged at revision: 48
Proposed branch: lp://staging/~stevanr/linaro-ci-dashboard/build-detail-view
Merge into: lp://staging/linaro-ci-dashboard
Diff against target: 125 lines (+68/-3)
6 files modified
dashboard/frontend/templates/build.html (+1/-1)
dashboard/frontend/templates/loop_build_detail.html (+17/-0)
dashboard/frontend/templates/loop_detail.html (+1/-1)
dashboard/frontend/urls.py (+3/-0)
dashboard/frontend/views/loop_build_detail_view.py (+45/-0)
dashboard/frontend/views/loop_build_view.py (+1/-1)
To merge this branch: bzr merge lp://staging/~stevanr/linaro-ci-dashboard/build-detail-view
Reviewer Review Type Date Requested Status
Milo Casagrande (community) Approve
Linaro Infrastructure Pending
Review via email: mp+124120@code.staging.launchpad.net

Description of the change

Build detail view implementation.

To post a comment you must log in.
49. By Stevan Radaković

Add schedule button fix.

Revision history for this message
Milo Casagrande (milo) wrote :

Hey Stevan, thanks for working on this!

It looks to go, just a couple of fixes from PEP8, can do that when merging:

loop_build_detail_view.py:27:1: E302 expected 2 blank lines, found 1
loop_build_detail_view.py:42:80: E501 line too long (120 characters)

review: Approve
50. By Stevan Radaković

pep8 fixes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches