juju / juju-gui-charm

Charm for Juju GUI.
GNU Affero General Public License v3.0
2 stars 16 forks source link

Remove now unnecessary apiUrl placeholder. #23

Closed jcsackett closed 8 years ago

jcsackett commented 8 years ago

QA:

JUJU_GUI_BRANCH=/path/to/the/PR make clean package
make deploy
juju set juju-gui sandbox=true
jujugui commented 8 years ago

Test PASSed. Refer to this link for build results (access rights to CI server needed): http://ci.jujugui.org:8080//job/juju-gui-charm/20/ Test PASSed.

hatched commented 8 years ago

:+1: :ok: All appears to work as expected in both the real env and in sandbox mode. Thanks for this cleanup!

huwshimi commented 8 years ago

Code :+1:

hatched commented 8 years ago

:shipit:

jcsackett commented 8 years ago

Thanks all. :shipit:

jujugui commented 8 years ago

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-charm-merge