Conversation
supportA very useful plugin but I had to hack it to get it to work on my setup. I have two suggestions 1. add filters around whether to use wp_mail() or mail(). I say this as if you filter wp_mail then the notification may never be sent. 2. If you added the functionality to set the options on mutisite at the network level that would be great. Id prefer this is the free version but if you put it in the pro version that would be understandable.
Hey @shawfactor , Thanks! 1. That’s a good idea. We can add a filter here, https://github.com/verygoodplugins/fatal-error-notify/blob/master/includes/class-public.php#L100 , will change that line to if ( function_exists( 'wp_mail' ) && apply_filters( 'fen_use_wp_mail', true ) ) { Then you can return false from the fen_use_wp_mail filter to have to use mail(). If that works for you? 2. That’s a nice idea as well. Will probably put it in the paid version though 😉 Jack
Hey @shawfactor , FYI we just released an update to the Pro version that enables network activating and configuring the plugin. Drop us a line at https://fatalerrornotify.com/support/contact/ and I’d be happy to send you a copy to try out. Thanks for the suggestion!
Thanks I’ve reached out Pete
Hey @shawfactor , Thanks! 1. That’s a good idea. We can add a filter here, https://github.com/verygoodplugins/fatal-error-notify/blob/master/includes/class-public.php#L100 , will change that line to if ( function_exists( 'wp_mail' ) && apply_filters( 'fen_use_wp_mail', true ) ) { Then you can return false from the fen_use_wp_mail filter to have to use mail(). If that works for you? 2. That’s a nice idea as well. Will probably put it in the paid version though 😉 Jack
Hey @shawfactor , FYI we just released an update to the Pro version that enables network activating and configuring the plugin. Drop us a line at https://fatalerrornotify.com/support/contact/ and I’d be happy to send you a copy to try out. Thanks for the suggestion!
Thanks I’ve reached out Pete