Added optional md5 as suffix to css/js

linnovate / meanio

meanio core module

MIT License

54 stars 73 forks source link

Added optional md5 as suffix to css/js #66

Closed jwebbdev closed 9 years ago

jwebbdev commented 9 years ago

Related to linnovate/mean#1293

This supports an extra object on mean configs of: assets : { hash: true, // default separator: '?v=' // default },

@liorkesos I'd appreciate input on config naming, and default settings, what you think makes sense in most cases and any problems you see with the approach. @dazwin Try this out and see if it fulfills the caching portion of your issue. We can look into extending this approach with liors suggestions in the issue for the cdn root.

dazwin commented 9 years ago

The approach looks good, but some assets get the hash whereas others don't...

bowercomponent/.css - no [module]/assets/css/_ - yes modules/rtl.css - no bowercomponents/.js - no [module]/_.js - no [module]/assets/lib/*.js - yes

I'm going to be on vacation next week, but I can start building the CDN deployment and test fully when I return.