18 react hot loader implementation #69
Conversation
18 ReactHotLoader/package.json
Outdated
| "redux-thunk": "^2.1.0", | ||
| "toastr": "^2.1.2" | ||
| "toastr": "^2.1.2", | ||
| "webpack-dev-server": "^1.14.1" |
There was a problem hiding this comment.
move webpack-dev-server to devDependencies
There was a problem hiding this comment.
Done, pending commit / push
18 ReactHotLoader/webpack.config.js
Outdated
| test: /\.(ts|tsx)$/, | ||
| exclude: /node_modules/, | ||
| loader: 'ts-loader' | ||
| loaders: ['react-hot', 'babel','ts-loader'] |
There was a problem hiding this comment.
I test to execute the app without babel-loader and it's working. So we only need:
loaders: ['react-hot', 'ts-loader']
There was a problem hiding this comment.
I think webpack works if we configure this line like:
loaders: ['react-hot-loader', 'ts-loader']
or
loaders: ['react-hot', 'ts']
It's the same for it
| @@ -0,0 +1,120 @@ | |||
| # React Hot Loader + Redux dev tool support | |||
There was a problem hiding this comment.
We were writing this info in AboutPage, isn't it?
There was a problem hiding this comment.
In the about page we can have a summary, but is a good idea to have detailed readme.md when a given user navigates to this sample in Github the readme.md will be automatically displayed
18 ReactHotLoader/package.json
Outdated
| "css-loader": "^0.23.1", | ||
| "deep-freeze": "0.0.1", | ||
| "enzyme": "^2.4.1", | ||
| "express": "^4.14.0", |
…re the dev server, removed devserver.js
4 participants
Adding react hot loading plus dev tool support.
Added as well a readme.md (feedback welcome on this document, is just barebones, we could add a lot of useful info here: e.g. links to hot loading, redux, dev tool, snapshots...).
TODO: I will open a new issue to proper create a dev configuration including react hot loading, but that's out of the scope for this case (a new case will be created).