Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

Out of all the patches the only ones that can easily be done is a merge of #1140356: Add async, onload property to script tags & #1664602: Allow attributes to be passed to drupal_add_[css|js] (SRI). This patch has been committed.

mikeytown2’s picture

Status: Active » Fixed
memoday’s picture

Hi Mike,

Is it OK to start using your D7 version on product site? Any recommendation/advice please?

mikeytown2’s picture

If you've tested it out locally and it works, it should be OK to use.

Just be aware of #1946218: test with other modules & themes. I especially need to make sure advagg works with the CDN and CSS Embedded Images modules. I'm pretty sure I have advagg working with jQuery Update. And some nice to have integration would be JS script loaders like LABjs, HeadJS, and Script.js.

I plan on releasing an alpha once CDN and CSS Embedded Images is working 100%. Until a 7.x-2.x alpha version is released there is no upgrade path (from dev to dev).

In terms of AdvAgg, it hasn't been thoroughly performance tested so it could be doing redundant stuff on the PHP side of things. Also the advagg_mod submodule has some experimental stuff in in, so I would be careful with what you do when using it.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.