-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/email verification fix #1839
Conversation
@brylie These is resovled issue. But I have the next information in meteor console:
|
Right, the mail settings have to be configured in order for the MAIL_URL environmental variable to be set. |
We have enabled verification emails at all times, which means that Meteor will sometimes try to send emails when the |
So, should I merge it? When MAIL_ENV is not set, email doesn't send to user. |
Jep. That is expected. We have to figure out how to dynamically enable the email related features, but in the meantime opted to enable them by default. |
@brylie I'm goint to merge it with a view to MAIL_ENV will be set and will work correct in the PR for 1843 issue |
Closes #1836
Changes