search

hotsh / rstat.us

Simple microblogging network based on the ostatus protocol.
http://rstat.us/
Other
722 stars 215 forks source link

Changing bio requires email action #735

Open burningTyger opened 11 years ago

burningTyger commented 11 years ago

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 commented 11 years ago

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 commented 11 years ago

yeahhhhh this seems wrong. thank you for reporting!!!! :heart:

zph commented 11 years ago

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