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 confirmation email option after customer creation #2164

Conversation

Alban-io
Copy link
Contributor

@Alban-io Alban-io commented May 26, 2016

Add confirmation email option after customer creation :

Todo List

  • Valid Token creation strategy
  • Update redirection in CustomerController
  • Add email customer confirmation

@gillesbourgeat gillesbourgeat added this to the 2.4.0-alpha1 milestone May 26, 2016
@Alban-io Alban-io force-pushed the feature/customer-creation-email-confirmation branch 2 times, most recently from 39ddff9 to b723aed Compare May 26, 2016 14:51
@Alban-io Alban-io force-pushed the feature/customer-creation-email-confirmation branch from b723aed to 9f4d968 Compare May 26, 2016 14:52
@Alban-io Alban-io force-pushed the feature/customer-creation-email-confirmation branch from 8fc1bba to c8357f4 Compare May 27, 2016 07:24
@Alban-io Alban-io force-pushed the feature/customer-creation-email-confirmation branch 3 times, most recently from f55a4af to 1a26b8a Compare May 27, 2016 14:51
@gillesbourgeat
Copy link
Member

Hello,
Have you finished this pull request ? The tag [WIP] is always present.

@Alban-io Alban-io force-pushed the feature/customer-creation-email-confirmation branch from 1a26b8a to 7cf7fea Compare May 30, 2016 07:27
@Alban-io Alban-io changed the title [WIP] Add confirmation email option after customer creation Add confirmation email option after customer creation May 30, 2016
@gillesbourgeat
Copy link
Member

Thanks !

@gillesbourgeat gillesbourgeat merged commit a751207 into thelia:master May 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants