enable webpack hmr

[jokester]

With this patch, modifying App.tsx will not cause a full page reload.

[lookfirst]

Seems a bit typesafe overkill, no?

This works for me...

const renderApp = () => {
	ReactDOM.render(<App />, document.getElementById('root') as HTMLElement);
};

renderApp();

declare const module: { hot: any };
if (module.hot) {
	module.hot.accept(['./App'], () => {
		renderApp();
	});
}

[deftomat]

@jokester Well, there is one missing thing: state will be not preserved between HMRs.
To make it works, you need to add react-hot-loader as described here.

So, please consider updating a webpack.config.dev.js file to make it works as expected (with preserving the state)

Also, without this, Apollo dev tool will also lost state after first HRM and you cannot inspect a watched queries.

[wmonk]

Hi, thanks for your PR! I am of the opinion that we shouldn't deviate too much away from the original react-scripts package, and have always found hot reloading to be pretty flaky, with people experiencing a whole host of issues. For this reason, I will leave it to be implemented by people ejecting, rather than a core part of this project.

[quantuminformation]

oh, I was so hoping for a way to do HMR without ejecting.

[kenzanboo]

@lookfirst suggestion works great!