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

Changing bio requires email action #735

Open
burningTyger opened this issue Dec 20, 2012 · 3 comments
Open

Changing bio requires email action #735

burningTyger opened this issue Dec 20, 2012 · 3 comments

Comments

@burningTyger
Copy link
Contributor

I changed my bio from nothing to something and got an email that tells me I need to reconfirm the changes to my email address (To confirm your e-mail change for rstat.us please click on the link below.) That's slightly irritating. I double checked I didn't change my address and assumed it's a bug.

@burningTyger
Copy link
Contributor Author

so I read through users_controller and it seems that the first time you edit your profile you get this email sent. Not so upon registration. Did I read that correctly?

edit: https://github.com/hotsh/rstat.us/blob/master/app/controllers/users_controller.rb#L68
create: https://github.com/hotsh/rstat.us/blob/master/app/controllers/users_controller.rb#L90

@carols10cents
Copy link
Contributor

yeahhhhh this seems wrong. thank you for reporting!!!! ❤️

@zph
Copy link
Contributor

zph commented Dec 28, 2012

Picking up from @burningTyger's comment, should it be in the create method after the if @user#save conditional @user.save ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants