Mathieu Lamiot
Mathieu Lamiot
Should be ready for review tomorrow
There are 2 defects preventing to happily run the LRC Beacon and related AJAX calls (so impacting #6874 & #6828). @jeawhanlee mentioned it should be a quick fix, he'll open...
Seems to be fixed with #6891 To be re-tested and confirmed.
https://github.com/user-attachments/assets/d3cf6a9d-9b25-4692-9d28-1382dbc5ac57
@Mai-Saad, about this: > Page with big HTML is handled => related https://wp-media.slack.com/archives/CUT7FLHF1/p1724324937592149?thread_ts=1724257215.870589&cid=CUT7FLHF1 You are seeing the hashes added to the DB, but there is a failed status with timeout...
Oh interesting. I suppose there are many hashes in the LRC table for this URL? PHP is complaining that the pattern we are looking for is too large here, likely...
Might be a duplicate of https://github.com/wp-media/wp-rocket/issues/6395
@Miraeld @jeawhanlee While this is an important first step, this is far from completing the front-end part of the feature. This issue seems mostly to be integrating the prototype in...
@jeawhanlee Thanks for raising the point. I feel like the missing part (applying the lazy render attribute) will be a complex thing, requiring a few days of work. On the...
I discussed this with @jeawhanlee and @Miraeld: the longer we can keep the 3 options (DOMDocument, Regex, SimpleDom) the better as it would allow us to switch to another if...