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.

Sign up for GitHub

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

Jump to bottom

Enable email verification #36

Closed
cdriehuys opened this issue Jul 11, 2017 · 2 comments
Closed

Enable email verification #36

cdriehuys opened this issue Jul 11, 2017 · 2 comments
Milestone
v0.4 - Account Ma...

Comments

@cdriehuys
Copy link
Member

Emails should be verified before a user can log in. Requiring a verified email would provide the following benefits:

  • If we move Friends over to this backend in the future, we would avoid conflicts with fake email addresses being duplicated across the two apps which would make an account merge very difficult.
  • Amazon SES (our transactional email provider) is very strict about how many emails we send that can bounce before we get classified as spammers. Requiring a verified email address would limit the number of emails we send to fake addresses.
@cdriehuys cdriehuys added this to the v0.4 - Account Management milestone Jul 11, 2017
@cdriehuys
Copy link
Member Author

Don't forget to check verification again when a user changes their email address.

@cdriehuys
Copy link
Member Author

We should remove the existing process for changing an email address and implement it as described in #38.

cdriehuys added a commit that referenced this issue Jul 12, 2017
@cdriehuys
Merge branch 'feature/email-validation' into develop
7043aec 
Email addresses now require verification before they can be used.

Closes #36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.4 - Account Management
Development

No branches or pull requests
1 participant
@cdriehuys