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

Fixed bug causing dropped charm zip files to be parsed as a bundle deployer file in IE11 #88

Merged
merged 2 commits into from Jan 27, 2014

Conversation

hatched
Copy link
Contributor

@hatched hatched commented Jan 27, 2014

To QA this is a long setup process of custom charms, juju etc. We can postpone it until putcharm is a little more stable across the ecosystem.

@jujugui
Copy link
Contributor

jujugui commented Jan 27, 2014

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

@jujugui
Copy link
Contributor

jujugui commented Jan 27, 2014

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

var file = {
// Using a complex name to make sure the extension filtering works
name: 'foo-bar.baz.zip',
// This MIME type is used in Chrome and Firefox, see the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammarian niggle: replace comma with semicolon or period.

@garyposter
Copy link
Contributor

👍 with ultra trivials.

@hatched
Copy link
Contributor Author

hatched commented Jan 27, 2014

Thanks for the review! :shipit:

@jujugui
Copy link
Contributor

jujugui commented Jan 27, 2014

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

jujugui added a commit that referenced this pull request Jan 27, 2014
To QA this is a long setup process of custom charms, juju etc. We can postpone it until putcharm is a little more stable across the ecosystem.
@jujugui jujugui merged commit abb5fe4 into juju:develop Jan 27, 2014
@hatched hatched deleted the local-charm-1273402 branch January 29, 2014 21:22
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