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

Emails and password reset #40

Closed
5 of 16 tasks
TimMikeladze opened this issue Jan 25, 2017 · 3 comments
Closed
5 of 16 tasks

Emails and password reset #40

TimMikeladze opened this issue Jan 25, 2017 · 3 comments
Assignees

Comments

@TimMikeladze
Copy link
Member

TimMikeladze commented Jan 25, 2017

The following should be implemented, additional details can be found here http://docs.meteor.com/api/passwords.html.

An important detail to note in the email implementation is that should be generic enough for consumers to plugin their own email service. For example using the mailgun api vs using their own smtp server.

Server

  • addEmail
  • removeEmail
  • sendResetPasswordEmail
  • sendEnrollmentEmail
  • sendVerificationEmail
  • resetPassword
  • sendWelcomeEmail

Server config

  • sendVerificationEmail (default false)
  • sendEnrollmentEmail (default false)
  • sendWelcomeEmail (default false)
  • email template for each email type, consider using https://github.com/mjmlio/mjml

Client

  • requestPasswordReset
  • requestVerificationEmail

Client config

  • onResetPasswordLink
  • onEnrollmentLink
  • onEmailVerificationLink
@pradel
Copy link
Member

pradel commented Feb 1, 2017

For the mail implementation I found this lib https://github.com/eleith/emailjs who looks good. I was using nodemailer but they changed their licence since v3

@pradel
Copy link
Member

pradel commented Feb 15, 2017

I am not sure we should provide email templates.
I think we could have only text emails and let the user do his own template.

@TimMikeladze
Copy link
Member Author

Sounds good to me.

@pradel pradel closed this as completed Feb 28, 2018
Aetherall pushed a commit that referenced this issue Mar 11, 2018
Aetherall pushed a commit that referenced this issue Mar 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants