erikras / react-redux-universal-hot-example

A starter boilerplate for a universal webapp using express, react, redux, webpack, and react-transform
MIT License
12k stars 2.5k forks source link

[question] Should Html use @connect? #1271

Open shaine opened 8 years ago

shaine commented 8 years ago

Howdy,

In reading through src/helpers/Html.js, I noticed that the Html component is receiving the store as an ownProp and referencing it directly in its render function. Would it be better to wrap Html in react-redux, and have the serialized store data get passed into Html as a prop? Or is there a good reason to not do that in this particular instance which I'm not seeing?

Thanks!