Mathieu Lamiot
Mathieu Lamiot
> add a method fetch_links that will send a request to the homepage, get the HTML, find the first 10 internal links, and send them to the jobs manager Total...
@Tabrisrp For this issue, can you clarify something please: does this implementation rely on a DB table? It seems to me that the suggested solution is that, on update/activation/install, we...
@mostafa-hisham, for the new WP Rocket version to be able to communicate properly with the SaaS, is it correct that the following changes should be made? - When sending a...
To whoever picks up this issue, it could make sense to divide it in 3 successive PRs: - First one to manage the fetch_link method on updatE/activation/install but without sending...
Yes, but in any case we should do this I think. Otherwise if we store the links somewhere, they would become outdated, no?
@Tabrisrp @Miraeld I think in that case we are missing a trigger in the current grooming: the fetch_link process should be triggered as well when clicking the "Clear Image Data"...
We have to trigger the fetch_link on some triggers like `wp_rocket_upgrade`, `rocket_after_activation`, as those cases won't be covered if we just trigger with the data deletion. For the fetch_link on...
Brought back to In Progress as there is a new AC: Bail out from this process when RUCSS is enabled. (Discussed [here](https://github.com/wp-media/wp-rocket/issues/6470#issuecomment-1988608525))
@jeawhanlee I just remembered there was already a dedicated issue for this accetpance criteria `Bail out from this process when RUCSS is enabled.`. It is here: https://github.com/wp-media/wp-rocket/issues/6435 Up to you,...
The only known issue to this day is ready to be merged: https://github.com/wp-media/wp-rocket/pull/6517 So let's merge it before running the above test plan. So that we only have to validate...