Skip to content

Validate email confirmation on sign up and profile update#1520

Closed
sonalkr132 wants to merge 2 commits into
rubygems:masterfrom
sonalkr132:email_confirmation
Closed

Validate email confirmation on sign up and profile update#1520
sonalkr132 wants to merge 2 commits into
rubygems:masterfrom
sonalkr132:email_confirmation

Conversation

@sonalkr132
Copy link
Copy Markdown
Member

Closes: #1512

Users will be required to retype their email in email confirmation field.
Sign up page - Profile update page
screenshot from 2016-12-22 14-36-30

@GBH
Copy link
Copy Markdown
Contributor

GBH commented Dec 22, 2016

Personally, I don't like this solution for these reasons:

Signup

  • During signup it's clunky to ask for email twice
  • If email is entered incorrectly it's not a big deal as user can try again. Side effect is that you end up with dead account. But that can be cleaned up easily by purging old unconfirmed accounts.

Profile update

  • Clunky again. If I want to change only my twitter handle it will force me to fill email address confirmation.
  • Nothing stops me from copy/pasting miss-typed email. That makes sense more for passwords (can't copy paste that)
  • If I end up copy/pasting bad email address I again successfully locked myself out from the account.

So with this it's still possible to lock yourself out. With #1518 it would be impossible to do that because you still can login back and fix error.

@sonalkr132
Copy link
Copy Markdown
Member Author

sonalkr132 commented Dec 23, 2016

Thank your for your feedback @GBH
I would suggest minor correction 😄

email is entered incorrectly it's not a big deal as user can try again

Username would no longer be available.

If I want to change only my twitter handle it will force me to fill email address confirmation.

You would only need to fill in email confirmation if you change your original email.

EDIT: Added caveat for sign up.

@sonalkr132
Copy link
Copy Markdown
Member Author

See #1531

@sonalkr132 sonalkr132 closed this Dec 31, 2016
@sonalkr132 sonalkr132 deleted the email_confirmation branch December 31, 2016 11:46
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.

2 participants