Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2857)

Issue 11295043: Subordinates don't get status bars

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by bcsaller
Modified:
10 years, 9 months ago
Reviewers:
jeff.pihach, mp+174842, gary.poster
Visibility:
Public.

Description

Subordinates don't get status bars Subordinates could render status in the future, but for now we omit this as the UI relating to mapping sub units to principal service units is unclear. https://code.launchpad.net/~bcsaller/juju-gui/nostatusforoldsubordinates/+merge/174842 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Subordinates don't get status bars #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -10 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/views/topology/service.js View 3 chunks +14 lines, -9 lines 0 comments Download
M app/views/utils.js View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 4
bcsaller
Please take a look.
10 years, 9 months ago (2013-07-15 19:42:57 UTC) #1
gary.poster
LGTM for now. :-) Thanks!
10 years, 9 months ago (2013-07-15 19:50:11 UTC) #2
jeff.pihach
LGTM!
10 years, 9 months ago (2013-07-15 20:57:21 UTC) #3
bcsaller
10 years, 9 months ago (2013-07-15 21:10:56 UTC) #4
*** Submitted:

Subordinates don't get status bars

Subordinates could render status in the future, but for now we omit 
this as the UI relating to mapping sub units to principal service units
is unclear.

R=gary.poster, jeff.pihach
CC=
https://codereview.appspot.com/11295043
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b