Display the new quickstart bundle deploy id on the deploy details page #697

Merged
merged 2 commits into from Feb 25, 2015

Conversation

Projects
None yet
4 participants
Owner

hatched commented Feb 25, 2015

Quickstart will support the new bundle id format very soon so this updates the deploy page on the bundle details page to this new id format.

Owner

hatched commented Feb 25, 2015

To QA

Visit the bundle details page for a promulgated and a namespaced bundle. On the "deploy" tab the quickstart deploy instructions should have id's in the format of bundle-version and u/username/bundle-version respectively.

Member

jujugui commented Feb 25, 2015

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

Member

jujugui commented Feb 25, 2015

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

Owner

hatched commented Feb 25, 2015

Spurious NPM failure

Owner

mitechie commented Feb 25, 2015

👍 no qa yet, will see if I can get an env up here shortly.

Member

makyo commented Feb 25, 2015

👍 No QA yet; thanks for the quick turn around.

Member

makyo commented Feb 25, 2015

QA okay! Everything looks good in the deploy tab. As discussed, we may want to revisit the "Location" link with UX at some point.

Owner

hatched commented Feb 25, 2015

Thanks for the reviews and qa's! :shipit:

Member

jujugui commented Feb 25, 2015

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

jujugui added a commit that referenced this pull request Feb 25, 2015

Merge pull request #697 from hatched/quickstart-deploys
Display the new quickstart bundle deploy id on the deploy details page

Quickstart will support the new bundle id format very soon so this updates the deploy page on the bundle details page to this new id format.

@jujugui jujugui merged commit ff4c889 into juju:develop Feb 25, 2015

1 check failed

default Merged build finished.
Details

@hatched hatched deleted the hatched:quickstart-deploys branch Mar 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment