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

Accounts.sendLoginEmail does not work if address is set #42

Closed
pfuerholz opened this issue May 11, 2016 · 3 comments
Closed

Accounts.sendLoginEmail does not work if address is set #42

pfuerholz opened this issue May 11, 2016 · 3 comments

Comments

@pfuerholz
Copy link

If parameter 'address' is set email's 'to'-address will not be set:

'To' is set with variable 'email':
var options = { to: email, from: Accounts.em

But 'email' is only set when address is not set:
if (!address) { var email = _.find(user.emails || [], function (e) { return !e.verified; }); address = (email || {}).address; }

This case happens when configuring 'EMAIL_ONLY_NO_PASSWORD':
Accounts.ui.config({ passwordSignupFields: 'EMAIL_ONLY_NO_PASSWORD', ...

  • Did I miss something?
  • Please tell me if you'd like a pull request.
@timbrandin
Copy link
Member

Good catch, we should probably send to address instead of email: 508aff3

timbrandin added a commit that referenced this issue May 24, 2016
@timbrandin
Copy link
Member

I've pushed a fix for this in 1.2.3, could you verify that it works?

@pfuerholz
Copy link
Author

Works for my use case. Thanks!

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

2 participants