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

Fix start script path so photon builds! 🙌🏻 #11

Merged
merged 2 commits into from
Oct 13, 2015

Conversation

ggamel
Copy link
Contributor

@ggamel ggamel commented Oct 13, 2015

This resolves the issue in #9.

See image of app after fix: photon-fixed

@ggamel ggamel mentioned this pull request Oct 13, 2015
@@ -18,7 +18,7 @@
"url": "https://github.com/connors/photon/issues"
},
"scripts": {
"start": "electron example-components"
"start": "electron dist/template-app/app.js"
Copy link
Owner

Choose a reason for hiding this comment

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

I think this can just be electron dist/template-app

@connors
Copy link
Owner

connors commented Oct 13, 2015

Whoops! Good catch. I just left one comment but otherwise that looks good.

@ggamel
Copy link
Contributor Author

ggamel commented Oct 13, 2015

@connors Just updated the PR, again. Works 100% on my end.

@connors
Copy link
Owner

connors commented Oct 13, 2015

Helllll yeah

connors added a commit that referenced this pull request Oct 13, 2015
Fix start script path so photon builds! 🙌🏻
@connors connors merged commit ada22d1 into connors:master Oct 13, 2015
@connors connors modified the milestone: v0.1.1 Oct 13, 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.

2 participants