search

smpallen99 / coherence

Coherence is a full featured, configurable authentication system for Phoenix
MIT License
1.27k stars 225 forks source link

Handle unconfirmed email #378

Closed klacointe closed 6 years ago

klacointe commented 6 years ago

Hello,

We are using coherence and we need to confirm user email on each update. We found an issue and saw that the feature is missing but PR are welcome: https://github.com/smpallen99/coherence/issues/74

Here is a PR that add an option to require email update confirmation instead of directly updating the database. We are using the unconfirmed_email attributes that was already present on the migration file (but not used in the library)

Can you please make a review on this?

Signed-off-by: Stéphane Robino stephane@wttj.co Signed-off-by: Kévin Lacointe kevinlacointe@gmail.com

smpallen99 commented 6 years ago

Thanks for the contribution!!