Skip to content
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

Add new action hook when user logs in #76

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mbarreiro85
Copy link

Hi there,

Thank you for your good work with this plugin, we at observador are happy to use it and want to make a contribution.

So as we make login from different sources, and we need to register which source was used to login, we created a action hook to know if the user that logged in came from your plugin or not.

If you need some modification from our part we are happy to do it.

kind regards,
Miguel Barreiro
https://observador.pt/

@rahul286 rahul286 requested a review from pdclark June 22, 2021 15:36
@ankitrox
Copy link
Contributor

@mbarreiro85 We have implemented the rewrite of the plugin where we are storing the meta information in usermeta table if user registers from the plugin.

https://github.com/rtCamp/login-with-google/tree/fix/oauth-class

https://github.com/rtCamp/login-with-google/blob/fix/oauth-class/src/Modules/Login.php#L218

@mbarreiro85
Copy link
Author

@ankitrox Thank you for your feedback, i will check it.

@mbarreiro85
Copy link
Author

@ankitrox, The pull request we made was about when users login with your plugin, not when they register, can you please take a look at it.

kind regards

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