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

Multi email invitable module needs to delegate to configured primary_email_method_name #18

Closed
clairity opened this issue Jun 18, 2017 · 5 comments

Comments

@clairity
Copy link

this line in the confirmable.rb model file needs to be updated to sync up with the newly configurable primary_email_method_name:

instead of

... to: :primary_email_record ...

it should be changed to

... to: Devise::MultiEmail.primary_email_method_name ...

if your method name is primary_email as is now recommended, then that line throws this error:

undefined local variable or method `primary_email_record' for #<User:0x00...>

changing it as above makes the error go away.

@allenwq
Copy link
Owner

allenwq commented Jun 19, 2017

@clairity Thanks a lot!

cc @joelvh

@joelvh
Copy link
Collaborator

joelvh commented Jun 19, 2017

Thanks for the heads up. This should be fixed in #16

@clairity
Copy link
Author

ok thanks. i need this fix to be able to deploy, so i might have to fork the repo in the meantime.

@joelvh
Copy link
Collaborator

joelvh commented Jun 19, 2017 via email

@allenwq
Copy link
Owner

allenwq commented Jun 27, 2017

@clairity the new version is released

@allenwq allenwq closed this as completed Jun 27, 2017
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

No branches or pull requests

3 participants