Closed (fixed)
Project:
Social Auth
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Jan 2017 at 15:59 UTC
Updated:
1 Feb 2017 at 02:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
nikathoneComment #3
gvsoMoving issue to social auth.
I added #2779641: Provide method to store user's picture as a reference too
Comment #4
nikathoneUpdated title for better description
Comment #5
nikathoneOops should say event dispatchers
Comment #6
nikathoneAnd here is a patch. I added a third event for the user fields are being initialized before creating the user entity. For example, the social auth facebook can use this to adds it's extras fields instead of using it's own create user. To test this patch you can add something like:
In your mymodule.services.yml and inside UserCreatedSubscriber.php
Comment #7
gvsoThanks @nikathone. This is an amazing job.
These are some of the many whitespaces in the patch.
Maybe we can rename the class to SocialAuthUserEvent
Isn't a way we can (optionally) pass the provider name to the events? For example Social Auth Google should pass the argument 'google', so Event Subscribers can use it to adapt their behavior.
Comment #8
nikathoneAdded a
plugin_idargument to be passed for when creating or login user through social auth. If this get committed then we will have to updatesocial_auth_googleto do something likein it's code.
Comment #10
gvsoThank you very much. I've committed the patch with a few changes.
Comment #11
gvsoSocial Auth Example was updated. We are only missing documentation for event handlers now