-
Notifications
You must be signed in to change notification settings - Fork 138
Closed
Labels
Good First IssueIssue particularly suitable to be worked on by new contributorsIssue particularly suitable to be worked on by new contributors[Type] EnhancementA suggestion for improvement of an existing featureA suggestion for improvement of an existing feature
Milestone
Description
Since the two modules for the Site Health checks for persistent cache and full page cache got merged into core, we need to add logic so that these two modules are only loaded going forward when the relevant core functionality is not already present. For that we should use the can-load.php approach which was established in #293.
Relevant WordPress core commits:
- https://core.trac.wordpress.org/changeset/53955
- https://core.trac.wordpress.org/changeset/54043
Since the class that has all the changes is only loaded within WP Admin, we may not be able to check for actual functions or methods reliably, as our plugin initialization would run too early. So potentially we will simply need to check that the WordPress version is lower than 6.1.
tillkruss
Metadata
Metadata
Assignees
Labels
Good First IssueIssue particularly suitable to be worked on by new contributorsIssue particularly suitable to be worked on by new contributors[Type] EnhancementA suggestion for improvement of an existing featureA suggestion for improvement of an existing feature