-
Notifications
You must be signed in to change notification settings - Fork 96
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
Crash when executing search #24
Comments
Thanks for the issue! Looks like the same issue I found for #21 where some values aren't defined. I previously made an update on the dev branch that at least catches the error so that app won't crash and I pushed that to the heroku site. It hasn't been merged to master yet though so that'll be why it's crashing your local. |
Thank you! I checked out the the dev branch and that does indeed fix it. I deployed my own heroku for now. I'll probably run into the same hourly cap at some point :/ Not directly related to this ticket but I found that I had to add NPM_CONFIG_PRODUCTION=false to the config otherwise the default heroku configuration pruned away the node_modules. Wondering if you saw that and if there's a better way to fix it. |
I have not seen that issue personally, everything worked when deploying on default heroku setup. However, someone else has issue in #15 where it looked like their modules didn't get installed, is this the same issue? |
Yes, that's exactly it. I will post my suggested fix there, and close this ticket as a dup of #21. Strange through. |
Youtube might have changed something yet again. Looks like the public heroku site is down, and likely because of the same crash I'm seeing locally. When executing a search, this exception brings down the server
The text was updated successfully, but these errors were encountered: