search

ctrlplusb / react-universally

A starter kit for universal react applications.
MIT License
1.7k stars 244 forks source link

upgrade dependencies. #561

Closed l-7-l closed 6 years ago

l-7-l commented 6 years ago

This PR mainly upgrades webpack and some dependencies

@ctrlplusb @hguillermo Please give me some advice.

l-7-l commented 6 years ago

wait: Replace webpack.optimize.UglifyJsPlugin to UglifyJsPlugin

hguillermo commented 6 years ago

@chenyang-biu close to finish your PR? I would like to try this boilerplate with react 16. If I can help let me know.

l-7-l commented 6 years ago

@hguillermo I'm sorry, I've been looking for work these days....

l-7-l commented 6 years ago

please enlighten.

birkir commented 6 years ago

Thanks for the PR but now the next branch has everything (and more).

sergiokopplin commented 6 years ago

👏

hguillermo commented 6 years ago

Thanks @birkir. Are you planning to merged next to master soonish?

birkir commented 6 years ago

We are waiting on one dependency to be released, react-tree-walker 👍

hguillermo commented 6 years ago

Perfect. I guess I will start from next and update it later.

unleashit commented 6 years ago

Just wondering... are the new updates to react-tree-walker going to support renderToNodeStream, or will the app still have to use renderToString first? Seems like one of the primary benefits to R16 is the ability to stream server rendering, which appreas to not be happening right now with this repo except at the end as a string.

hguillermo commented 6 years ago

@birkir Do you know when are you merging next to master? What are the pending things if I use next as a base for my next project? Thanks

birkir commented 6 years ago

I do not. My buddy, the project owner, @ctrlplusb is the man to ask! 👍

hguillermo commented 6 years ago

Thanks @birkir.

@ctrlplusb: when do you think you will merge next to master? is it something pending?