Skip to content

Conversation

shawnhooper
Copy link

This plugin addresses issue #5.

In the process, I also created a generic is_user_disabled() function that checks the usermeta value. This was being done in two places already, and this PR added a third.

@ebauch
Copy link

ebauch commented Dec 3, 2018

Will this work if you replace Wordpress notifications with another plugin such as Better Notifications for WordPress?

@shawnhooper
Copy link
Author

@ebauch I just looked at Better Notifications for WordPress' code quickly, and I believe it is compatible. However, you should look at pull request 15 (#15 (comment)) instead. It's the better version.

@ebauch
Copy link

ebauch commented Dec 6, 2018

@shawnhooper thanks for your quick response and support. I can confirm that this does not work with Better Notifications atm. It may be related to being a MU site. I deactivated users as Network Admin and they do show up as disabled in all sub-wordpress blogs. However, notifications are still being sent to all users. It may also be related of me using Post SMTP plugin and not the Wordpress mailer.
So it either doesn't work because of Better Notifications plugin, MU site, or the Post SMTP plugin.
Because #15 is not approved, not sure what to do next.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants