Piotrek

Results 322 comments of Piotrek

Acceptance Criteria - `rocket_clean_post` should be guarded against clearing the whole cache (possible relation https://github.com/wp-media/wp-rocket/issues/5887) - Clear the cache of posts that contain updated Elementor module - No regressions in...

@jeawhanlee @engahmeds3ed The effort here is [M], not [S], right?

@DahmaniAdame Just to be sure, it's about clearing the Used CSS of related posts, right?

@DahmaniAdame I think this should be limited to specific templates only. As per @wp-media/qa and I also confirmed this on my website, if you edit the regular template using free...

@valmirselmani Yes, I can see that. For now you could modify the following helper plugin to achieve what you want: https://github.com/wp-media/wp-rocket-helpers/blob/master/cache/wp-rocket-no-cache-auto-purge/wp-rocket-no-cache-auto-purge.php

@DahmaniAdame The issue initially was about the all templates, was surprised while checking the QA report + doing some research 😁 Sending it back to QA then. Do you have...

I'm blocking this one for now, it needs a product discussion a bit, will move it back to grooming as soon as we have all the answers.

@DahmaniAdame This issue became more complicated than we thought it'll be. To avoid releasing important regressions, I'd suggest adding to `AC`: 1. Clear the whole cache when display conditions are...

@DahmaniAdame Are we okay with this part? ``` If we introduce the above, there'll be regression related to not clearing cache of the Templates with Conditions, https://github.com/wp-media/wp-rocket/issues/5848#issuecomment-1545701597 ``` We could...

To follow-up on this one, with new acceptance criteria. We'll preserve the current functionality of this PR: 1. `rocket_clean_post` should be guarded against clearing the whole cache :heavy_check_mark: 2. Clear...