Client side doesn't have hot reload

[shavyg2]

let me know if this is the correct repo to post this or if i should move it to the template

https://github.com/sveltejs/sapper/blob/master/sapper-dev-client.js#L6

module.hot is undefined.

I think this is because of this

https://github.com/sveltejs/sapper-template/blob/79be87c3f5cb90ee1afb2561965597f9e4273d26/webpack.config.js#L25

[Conduitry]

As noted in the comment, webpack HMR isn't working in Svelte v3 pending that issue and some hooks provided by the compiler to enable it.

[shavyg2]

Ye I am saying that the JavaScript errors because hot is not defined. Are you saying it doesn't matter because there is no hot module reloading regardless?

[jcollum]

He's saying use rollup instead of webpack until they fix that bug sveltejs/svelte#2377

But I started my project with webpack so now I have to translate to rollup 😢

[TrungRueta]

@jcollum you can just use webpack but disable hot-replacement feature in sapper.
open package.json, at line dev: "sapper dev" change to dev: "sapper dev --no-hot" to disable hot reload.
Sapper will auto use live reload when hot reload disabled, and from now on it work 100% same as rollup setup.

[jcollum]

Getting rollup going wasn't any big deal.

[TrungRueta]

rollup has not hot-replace feature, it only has live-reload, so issue not raised in rollup like webpack

[antony]

Closing this since there is an active effort on Svelte to get HMR working which will solve this.