Included Health Check Plugin

  • Resolved bigdawggi

    (@bigdawggi)


    5 years, 12 months ago

    Hi there, I’m looking at this plugin before installing it on a client’s site, and I’m wondering why you’ve included the entirety of the Health Check Plugin inside your own plugin? At a cursory glance, it looks like you spin up the plugin in the constructor, but it’s never utilized. This seems like an easy way for the included plugin to get out of date, and potentially create conflicts with someone who’s already installed the Health Check Plugin at a different version.

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author NSquared

    (@croixhaug)

    5 years, 12 months ago

    Hi there, thanks for the message. We are using those functions for our “Support” tab of SSA to help with troubleshooting. We have made some modifications and also changed class names to avoid any conflict. We’ve made enough modifications that we will be maintaining it separately and adding SSA-specific health checks (checking our database tables, notification system, etc.) into our Support tab. The code inside Health Check was a good starting point for us, but we will be taking it a slightly different direction in the future.

    Thanks,
    Nathan

    Thread Starter bigdawggi

    (@bigdawggi)

    5 years, 11 months ago

    Sounds great, thanks for the quick reply!

    Nice plugin btw 😉

    • This reply was modified 5 years, 11 months ago by bigdawggi. Reason: marking resolved
    Plugin Author NSquared

    (@croixhaug)

    5 years, 11 months ago

    Thank you 🙂

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Included Health Check Plugin’ is closed to new replies.