Merge lp://staging/~gary/juju-gui/trivialRelationMessage into lp://staging/juju-gui/experimental

Proposed by Gary Poster
Status: Merged
Merged at revision: 1044
Proposed branch: lp://staging/~gary/juju-gui/trivialRelationMessage
Merge into: lp://staging/juju-gui/experimental
Diff against target: 10 lines (+3/-0)
1 file modified
app/templates/service-relations-list.handlebars (+3/-0)
To merge this branch: bzr merge lp://staging/~gary/juju-gui/trivialRelationMessage
Reviewer Review Type Date Requested Status
Juju GUI Hackers Pending
Review via email: mp+185693@code.staging.launchpad.net

Description of the change

Add message if no relations exist

https://codereview.appspot.com/13253054/

To post a comment you must log in.
Revision history for this message
Gary Poster (gary) wrote :

Reviewers: mp+185693_code.launchpad.net,

Message:
Please take a look.

Description:
Add message if no relations exist

https://code.launchpad.net/~gary/juju-gui/trivialRelationMessage/+merge/185693

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/13253054/

Affected files (+5, -0 lines):
   A [revision details]
   M app/templates/service-relations-list.handlebars

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision: <email address hidden>

Index: app/templates/service-relations-list.handlebars
=== modified file 'app/templates/service-relations-list.handlebars'
--- app/templates/service-relations-list.handlebars 2013-09-13 21:08:43
+0000
+++ app/templates/service-relations-list.handlebars 2013-09-16 00:32:31
+0000
@@ -1,3 +1,6 @@
+{{#unless relations}}
+<h2>This service has no relations.</h2>
+{{/unless}}
  {{#relations}}
    <h3>{{far.service}}</h3>
    <h4>Interface: {{interface}}</h4>

Revision history for this message
Gary Poster (gary) wrote :

*** Submitted:

Add message if no relations exist

R=
CC=
https://codereview.appspot.com/13253054

https://codereview.appspot.com/13253054/

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