Deploying to heroku does not work #221

Closed
dgollahon opened this Issue Jul 25, 2016 · 8 comments

Comments

Projects
None yet
7 participants

dgollahon commented Jul 25, 2016 edited

First of all, thanks for building this--I've found it useful. Unfortunately it seems like directly deploying to heroku does not work correctly.

It appears to be due to this line, which expects dist/ to exist, but does not by default, since it is ignored from source control.

I was able to work around it, but it was pretty confusing when I first tried to setup the server. It would be great if you could fix this or, at least, provide a better warning when unable to access dist/.

Thanks!

tonyedwardspz commented Jul 25, 2016 edited

@dgollahon I'm having the same issue. How did you work around it? Build locally and then deploy to heroku?

@tonyedwardspz: yeah, just checked dist/ into my copy of the repo and pushed that.

You can use this if you need to in the meanwhile:

https://heroku.com/deploy?template=https://github.com/niftylettuce/slackin/tree/master

Confirmed that @niftylettuce's branch works fine. This bug has caused a LOT of teeth gnashing. Thank you @niftylettuce for the help!

duaneking commented Aug 13, 2016 edited

When will this be fixed in the official branch?

productcampsg commented Aug 16, 2016 edited

Thank you @niftylettuce! I had the Application Error page whenever I tried deploying to Heroku using the official branch.

I'm getting a 404 on http://slack.productcampsg.com/ but http://productcampsg.herokuapp.com shows the app.

Something like http://slack.mindtheproduct.com/ works. Does the Landing Page work for all of you 👀?

@productcampsg it works for me - maybe your Heroku dyno needs to be upgraded to Hobby or have a server ping it occasionally to wake up

aaalveee commented Dec 7, 2016

@niftylettuce's branch is crashing for me, shows "Application Error"

rauchg closed this in 77c9d94 Feb 9, 2017

@mexisme mexisme added a commit to devopsnz/slackin that referenced this issue Mar 10, 2017

@rauchg @mexisme rauchg + mexisme Include dist in deploy. Fix #221. (#223) 457c855

@mexisme mexisme added a commit to devopsnz/slackin that referenced this issue Mar 10, 2017

@rauchg @mexisme rauchg + mexisme Include dist in deploy. Fix #221. (#223) eeff657
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment