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

Issue 11301043: Expose showGetJujuButton Juju GUI setting

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by j.c.sackett
Modified:
10 years, 9 months ago
Reviewers:
rharding, mp+174859, gary.poster
Visibility:
Public.

Description

Expose showGetJujuButton Juju GUI setting This setting changes the "Logout" button to a "Get Juju" link; used only for the eventual jujucharms.com deployment. https://code.launchpad.net/~jcsackett/charms/precise/juju-gui/expose-juju-button/+merge/174859 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Expose showGetJujuButton Juju GUI setting #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -4 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M config.yaml View 1 chunk +7 lines, -0 lines 0 comments Download
M config/config.js.template View 1 chunk +2 lines, -1 line 0 comments Download
M hooks/backend.py View 1 chunk +3 lines, -1 line 0 comments Download
M hooks/utils.py View 2 chunks +3 lines, -2 lines 0 comments Download
M tests/test_utils.py View 1 chunk +11 lines, -0 lines 0 comments Download

Messages

Total messages: 4
j.c.sackett
Please take a look.
10 years, 9 months ago (2013-07-15 20:46:27 UTC) #1
rharding
LGTM
10 years, 9 months ago (2013-07-15 20:50:23 UTC) #2
gary.poster
LGTM
10 years, 9 months ago (2013-07-15 20:57:16 UTC) #3
j.c.sackett
10 years, 9 months ago (2013-07-16 12:13:43 UTC) #4
*** Submitted:

Expose showGetJujuButton Juju GUI setting

This setting changes the "Logout" button to a "Get Juju" link; used only for the
eventual jujucharms.com deployment.

R=rharding, gary.poster
CC=
https://codereview.appspot.com/11301043
Sign in to reply to this message.

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