New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Divi compatibility: show admin notice to clear the used CSS when a template is updated #6074
Divi compatibility: show admin notice to clear the used CSS when a template is updated #6074
Conversation
…ates_clear_rucss' into enhancement/5837-divi_save_templates_clear_rucss
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No unit/integration tests?
I forgot the tests, we have two public methods, going to add the tests for them now. |
tests/Unit/inc/ThirdParty/Themes/Divi/handleDiviAdminNotice.php
Outdated
Show resolved
Hide resolved
tests/Unit/inc/ThirdParty/Themes/Divi/handleDiviAdminNotice.php
Outdated
Show resolved
Hide resolved
tests/Unit/inc/ThirdParty/Themes/Divi/handleDiviAdminNotice.php
Outdated
Show resolved
Hide resolved
@engahmeds3ed Thanks for the PR. |
…r_rucss # Conflicts: # inc/Engine/WPRocketUninstall.php
All done now |
@engahmeds3ed Thank you for the PR. There are a couple of scenarios that are failing. With Remove Unused CSS enabled and used CSS existing in the database:
For the second scenario, with Remove Unused CSS enabled and used CSS existing in the database:
Can you please look into these? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working as expected. 👍
TestRail report: testrail-report-514.pdf
…ates_clear_rucss' into enhancement/5837-divi_save_templates_clear_rucss
Description
Show admin notice to clear the used CSS when a template is updated only when this template is used in any (page, post, tag, ...etc.)
Also added a new filter `` to enable/disable this functionality.
Fixes #5837
Type of change
Is the solution different from the one proposed during the grooming?
Yes it's changed a bit, you can check the discussion in the issue itself for more details.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Please delete the options that are not relevant.