-
Notifications
You must be signed in to change notification settings - Fork 89
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
Handle registration for exist customer #546
Handle registration for exist customer #546
Conversation
What happens if I enter an already existing email can I change the whole customer with that. This would mean I can just "register" with my friends email address and take over his account. |
@mamazu like in Sylius you first need to click on the Confirmation Link in the email to enable the new created account so without access to the Email account you will not be able to login or see anything in this account. |
@alexander-schranz |
@robmeek Thats a good point! Sadly this is what sylius currently also does when you do it over the normal sylius standard UI I think this should then also be changed there. //cc @lchrusciel |
Totally makes sense to add this behavior. As close as possible to Sylius I would say. It could be considered as a bug even. |
fc35642
to
271f968
Compare
271f968
to
df65705
Compare
Fixes #533 |
Thanks, Alexander! 🥇 |
Closes #533
This would handle the registration process like Standard Sylius that a Customer is set to the User if the E-Mail Address exist as a customer. Currently you are not able to register if you did an anonymous order over the shop api.