Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

View compose yml #526

Merged
merged 1 commit into from
Jun 11, 2015
Merged

View compose yml #526

merged 1 commit into from
Jun 11, 2015

Conversation

dharmamike
Copy link
Collaborator

This change adds the ability to view the Docker Compose representation of a Panamax application.

@@ -31,6 +31,8 @@

$('#manage_flow section.applications').applicationActions();

$('#manage_flow section.deployment-details').composeExportDialog();
Copy link
Contributor

Choose a reason for hiding this comment

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

It may be more clear to init this the same way in both cases. E.G.

$('#manage_flow section.deployment-details, #manage_flow section.applications').composeExportDialog();

And kill line 26 in jquery.application_actions.js

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh NM, this is a different component

@alexwelch
Copy link
Contributor

👍

1 similar comment
@rheinwein
Copy link
Collaborator

👍

@dharmamike dharmamike force-pushed the feature/view-compose-yml branch from 6065674 to 29eb8b1 Compare June 11, 2015 16:04
@dharmamike dharmamike merged commit 29eb8b1 into master Jun 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants