Merge lp://qastaging/~doanac/uci-engine/webui-pause into lp://qastaging/uci-engine

Proposed by Andy Doan
Status: Merged
Approved by: Andy Doan
Approved revision: 593
Merged at revision: 594
Proposed branch: lp://qastaging/~doanac/uci-engine/webui-pause
Merge into: lp://qastaging/uci-engine
Diff against target: 142 lines (+85/-4)
6 files modified
webui/common/static/common/paused.svg (+15/-0)
webui/tickets/static/tickets/webui.js (+12/-4)
webui/tickets/static/tickets/webui_review.js (+34/-0)
webui/tickets/templates/tickets/review.html (+17/-0)
webui/tickets/urls.py (+2/-0)
webui/tickets/views.py (+5/-0)
To merge this branch: bzr merge lp://qastaging/~doanac/uci-engine/webui-pause
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Joe Talbott (community) Approve
Review via email: mp+223463@code.qastaging.launchpad.net

Commit message

webui: add a read-only "reviews" page

The lander now has the ability to "pause" a ticket. A paused ticket
can mean one of two things:

1) the ticket needs one or more reviews performed
2) the ticket needs to retry a step

This handles displaying a ticket "needs attention" and provides a link
to see what actions need to happen. For now it just shows the missing
actions. The next step will be to add buttons to allow the ticket to continue.

Description of the change

Adds smarts to the webui to display a ticket that has been paused.

ticket-view related GUI change:
 http://i.imgur.com/q06OZ9H.png

new review page:
  http://i.imgur.com/0Y9BdUD.png

To post a comment you must log in.
Revision history for this message
Joe Talbott (joetalbott) wrote :

LGTM

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:593
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/880/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/880/rebuild

review: Approve (continuous-integration)

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