-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
include the core plugin files on the login page #135
base: master
Are you sure you want to change the base?
Conversation
ping @pitbulk |
Yes please merge this PR, we need this fix for upgraded version where it is breaking. |
Im trying to get privileges again on this and rest of SAML repos |
Any progress on this? |
@tomasvanagas - Still working on getting @pitbulk access. I wish I had a better answer 😞 |
@mrmoss any progress giving me access to WP plugin at Github and Wordpress site? |
I've filed a similar PR to address where the function does not need to exist and instead relies on the This fixes the issue with:
Hope this helps! |
Issue: #129
Fixes the missing function error by including the WordPress core functions. I am not clear why newer versions of WP are breaking, but the root of the issue comes down to the plugin firing the
is_plugin_active
function before WordPress has loaded it.