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

Validate change of email address request #247

Closed
dpslwk opened this issue Aug 29, 2017 · 3 comments
Closed

Validate change of email address request #247

dpslwk opened this issue Aug 29, 2017 · 3 comments
Assignees
Labels

Comments

@dpslwk
Copy link
Member

dpslwk commented Aug 29, 2017

Best practice would be to confirm the new email address and also send an email to the old address with details on how to recover account if it wasn't them

@dpslwk dpslwk added this to the 2.1 Additional features milestone Aug 29, 2017
@dpslwk
Copy link
Member Author

dpslwk commented Aug 29, 2017

currently this can be edited by the user.edit view

@dpslwk
Copy link
Member Author

dpslwk commented May 20, 2018

We should also confirm user email upon account creation (event Illuminate\Auth\Events\Registered)

suggest a new listener App\Listeners\Membership\ConfirmEmailOnUserRegistered

once confirmed we can go through something like (moving membership team approval to after confirmation)

'App\Events\Membership\NewUserEmailConfirmed' => [
    'App\Listeners\Membership\ApprovalEmailOnUserRegistered'
],

@dpslwk dpslwk self-assigned this May 20, 2018
@dpslwk
Copy link
Member Author

dpslwk commented Sep 23, 2018

5.7 has something that might help get started on this
https://laravel.com/docs/5.7/verification

dpslwk added a commit that referenced this issue Dec 30, 2018
# Conflicts:
#	resources/views/home.blade.php
dpslwk added a commit that referenced this issue Dec 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant