Skip to content

Fix customer login event#13

Open
mmenozzi wants to merge 2 commits intomailup:masterfrom
mmenozzi:patch-2
Open

Fix customer login event#13
mmenozzi wants to merge 2 commits intomailup:masterfrom
mmenozzi:patch-2

Conversation

@mmenozzi
Copy link
Contributor

When the customer_customer_authenticated event is fired the customer has not been put in session yet. This could lead to problems with subsequent observers that assume that the customer is logged in.
The customer_login event is instead fired just after the customer has been put in session.

When the customer_customer_authenticated event is fired the customer has not been put in session yet. This could lead to problems with subsequent observers that assume that the customer is logged in.
The customer_login event is instead fired just after the customer has been put in session.
@mmenozzi mmenozzi closed this May 11, 2022
@mmenozzi mmenozzi reopened this May 11, 2022
@mmenozzi mmenozzi changed the title Change customer login event Fix customer login event May 11, 2022
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.

1 participant