gocodebox / lifterlms

LifterLMS, a WordPress LMS Solution: Easily create, sell, and protect engaging online courses.
https://lifterlms.com
GNU General Public License v3.0
180 stars 134 forks source link

Transition System Report to WP Core "Site Health Info" screen #854

Open thomasplevy opened 5 years ago

thomasplevy commented 5 years ago

With the introduction of WP_Debug_Data powering the Tools -> Site Health -> Info screen we can transition from having our own system report screen and functionality and use the core methods and api.

Use WP_Debug_Data::debug_data()

Namely the filter debug_information

Remove from LifterLMS System Report (as they're now redundant)

Add to the Core Site Health Report

Additional

New in 5.8: https://make.wordpress.org/core/2021/06/22/extending-the-site-health-interface-in-wordpress-5-8/

thomasplevy commented 2 years ago

@pondermatic's question about database tables in the system report (which are present in the site-health screen) got me thinking we should prioritize this. It's not anything that anyone will really care about but it is a small improvement that will

A) Help us reduce our code footprint by getting rid of redundant code (server and wp core config stuff that already exists in site health provided by the WP core) B) By default provide us with a bunch of additional information C) I don't know if other plugins are jumping on this yet, but, if they do, it will also provide us with their data automatically if they add it (which could help us in conflict debugging)