Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Fixed bug 1321345 #335

Merged
merged 1 commit into from May 20, 2014
Merged

Fixed bug 1321345 #335

merged 1 commit into from May 20, 2014

Conversation

hatched
Copy link
Contributor

@hatched hatched commented May 20, 2014

When charm urls contained a hyphenated name and a charm version number the package name was not parsed correctly when creating a new model.

To QA

  • Use the il and mv flags (This bug only presented itself using these flags).
  • Search for the juju-gui.
  • Drag and drop it onto the canvas.
  • Click the service icon to open the inspector.
  • Close the inspector
  • The service token should still read juju-gui

@jcsackett
Copy link
Contributor

QA ok, code looks fine. 👍

@hatched
Copy link
Contributor Author

hatched commented May 20, 2014

Thanks for the review and qa! :shipit:

@jujugui
Copy link
Contributor

jujugui commented May 20, 2014

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

@jujugui
Copy link
Contributor

jujugui commented May 20, 2014

Test PASSed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/1058/

jujugui added a commit that referenced this pull request May 20, 2014
When charm urls contained a hyphenated name and a charm version number the package name was not parsed correctly when creating a new model.

#### To QA
- Use the il and mv flags (This bug only presented itself using these flags).
- Search for the `juju-gui`.
- Drag and drop it onto the canvas.
- Click the service icon to open the inspector.
- Close the inspector
- The service token should still read `juju-gui`
@jujugui jujugui merged commit 2e8d73e into juju:develop May 20, 2014
@hatched hatched deleted the inspector-1321345 branch May 22, 2014 17:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants