support webpack 4.8+

[outshineamaze]

1. modify : return _.replace(""CHUNK_MANIFEST"",..... line 63
2. add compilation.mainTemplate.hooks.assetPath ..... line 33

[dangreaves]

This would be super useful to get merged 🚀

[kishankanugula]

Do we have any alternatives until this is merged?

[niemyjski]

@diurnalist @bruderstein @spadgos @shakyShane @brandondoran @matthiasfeist @gingur @soundcloud/web-collective, @janizde and @MarinaFroes Can we get someone to look at this pr (or #56 #63, #64) or add some new maintainers... Lots of people blocked and no communication :\

[bruderstein]

I can see if we can get this merged. I'll also try and update the status - we no longer use this internally, as contenthash is now significantly better than it used to be.

[bruderstein]

@niemyjski are you able to confirm that this works in 4.8+?

[niemyjski]

@bruderstein Thanks, I just started to look at the various pr's to figure out which one is the best one and then I was going to test.

Thanks for the reply, Makes me wonder if we should update the readme to state that with a snippet or steps to upgrade to content hash. I have a large project that is out of date and I'm just trying to get it to work on latest before I go changing things out.

[bruderstein]

Just a quick update - I'd got two plugins mixed up when I said "we no longer use this internally, as contenthash is now significantly better than it used to be." - sorry, doing too many things concurrently!

We do no longer use this, but I think it's because since webpack 4, module_ids are more stable, so things are generally better. Webpack 5 improves this further with hashed module_ids, making them more stable.