search

mperdeck / dynamicbundles

Dependency resolver for MVC web sites, using MVC bundles
Apache License 2.0
16 stars 7 forks source link

Name the asset as a bundle for caching purposes #2

Open sz1149 opened 9 years ago

sz1149 commented 9 years ago

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

mperdeck commented 9 years ago

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.

RudeySH commented 7 years ago

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.

MrCsabaToth commented 7 years ago

Will this be merged? Sounds like a performance improvement.

RudeySH commented 6 years ago

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

MrCsabaToth commented 6 years ago

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