dynamicbundles icon indicating copy to clipboard operation
dynamicbundles copied to clipboard

Name the asset as a bundle for caching purposes

Open sz1149 opened this issue 10 years ago • 5 comments

I like what you have done with DynamicBundles and am planning on using it in a project; good work.

I thought an enhancement would be to add /bundle/ to the path. The reason is so response headers on /bundle/ can be added to enable browser caching resulting in faster page load time. I tested this by adding cache control rules in IIS on the /bundle/ path and was pleased with the result.

Let me know your thoughts.

Thank you, Stuart

sz1149 avatar Jul 28 '15 15:07 sz1149

Thanks for your pull request. I'm glad you like the idea behind (my take on) dynamic bundles.

I am interested in your pull request and your thinking behind it. However, I'm overseas at the moment. I'll look at it when I'm back home mid September.

mperdeck avatar Aug 02 '15 02:08 mperdeck

Will this ever be merged? If so, I would suggest to use "~/bundles/" with an "s", that seems to be the default naming convention for bundles in ASP.NET.

RudeySH avatar Jan 27 '17 14:01 RudeySH

Will this be merged? Sounds like a performance improvement.

MrCsabaToth avatar Mar 24 '17 03:03 MrCsabaToth

For future readers: I forked dynamicbundles a while ago and merged this pull request over there. https://github.com/RudeySH/dynamicbundles

RudeySH avatar Oct 10 '18 20:10 RudeySH

@RudeySH Oh yes, and yours is available as a NuGet package as well also. I highly recommend to anyone!

MrCsabaToth avatar Oct 11 '18 00:10 MrCsabaToth