-
Notifications
You must be signed in to change notification settings - Fork 356
Errors during first request of development server #73
Comments
I have the same exact behavior, I'll try to pull from a previous commit to see if symptoms change. |
@scarmuega circling back on this one. I roughly figured out the problem as it pertains to me. |
Finally, I found the error. This is what happens when you update React and don't update Helmet. Goja does not show the error line in bundle.js which makes debugging process very hard. Isn't there an easier way for this? |
@ufukomer, great! Could you send a PR with updated deps? |
I was trying to update all the deps as well but it took me way too long and resigned. Anyway, make sure to temporarily turn off server side rendering when you update dependencies, it'll make it easier to track down all the issues. |
@iKonrad If you disable server rendering you won't get most of these errors. 😁 Let's turn this issue into print stack trace of goja errors. That is a goja issue lol. Check this out: dop251/goja#28 (comment) Though it does not help much but better than none. @olebedev I made lots of changes. I need to fork and update from scratch. So I can do it in the future, not today. By the way, I have not migrated to React Router v4. It is very complex to me for now. There are lots of changes in API. For example, no more onEnter method of a router. So we should fetch in the componentDidMount for client side. Therefore that requires changes on server code too, something like passing init values (store's initialState -> config values in our app) into template directly from Go instead of fetching it during the server render. Am I right? |
It looks like
make serve
starts everything up nicely but upon the first request the go server panics and throws a TypeErrorI can't for the life of me figure out where this is going wrong.
make install
works without any errors, evenmake build
works, but upon running the server the error is thrown.I would love to play with this project a little bit. If someone has any thoughts please let me know.
The text was updated successfully, but these errors were encountered: