Leonardo Tegon

Results 17 comments of Leonardo Tegon
trafficstars

Hi @pmichaeljones, thanks for reporting this. I tried to reproduce the issue using a newly created application but I couldn't - I only received the confirmation email. [email-changed-signup.zip](https://github.com/plataformatec/devise/files/3009890/email-changed-signup.zip) I'm attaching...

Sorry @pmichaeljones it's been really busy at work lately, I wasn't able to do much open source 😞 I'll try to get back to you this week.

Yeah, I just wanted to understand exactly what's happening before thinking of a solution. It seems weird to me that the `email_changed?` method returns true since your code only updated...

Hi everyone, thanks for the feedback. Although I can see the value of this, I'm not sure it's a good idea to add a configuration option to support it. If...

@kylefox > With all due respect, I think you might be worrying about a problem you don't have 😇 Can you defer deciding whether (or not) to support options for...

@kylefox thanks for clarifying the use case for this. It makes a lot more sense now 👍 I agree with the index part, the best we can do is to...

@hudakh The [opened PR](https://github.com/plataformatec/devise/pull/4793) needs tests and some other changes before it can be merged.

@xhocquet @flynfish thanks, I've added it to my list and will review it when I can. Sorry for the delay.

@xhocquet Unfortunately, I don't have an ETA. I'm struggling to find the time to work on OSS lately, but since a lot of you are asking for this feature I'll...

@feliperenan Great work on this! My 2 cents here is that since this only happens in the test environment - e.g. because the session is memoized - I believe there's...