dc7290 / template-ejs-loader

ejs-loader with webpack5 support. Chain it to html-loader and use it with html-webpack-plugin.
MIT License
24 stars 3 forks source link

Fix problem: webpack-dev-server seems to ignore the change of ejs partial template which is "nested" in another ejs partial template. #13

Closed mizok closed 2 years ago

mizok commented 2 years ago

Fix #12

Hey, I found that the problem was in the function getIncludeEjsDependencies, actually you had to make it recursive to get all those nested template to be dependencies.

So I took a try, and it seemed to work 😆

Would you please check it ?

I REALLY need this feature BTW. 😢

dc7290 commented 2 years ago

@mizok

Thanks for creating the pull request!

I tried it in my local environment and it looks fine, so I will merge it. Thanks for the worthwhile commit.

mizok commented 2 years ago

@mizok

Thanks for creating the pull request!

I tried it in my local environment and it looks fine, so I will merge it. Thanks for the worthwhile commit.

NIce, thank you man XD Have a nice day !