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

feat(): #OIC-31 Add Smart Mail service #170

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

ClemLcs
Copy link

@ClemLcs ClemLcs commented Aug 27, 2023

Describe your changes

  • Add a service allowing you to choose between the sendinblue or nodemailer mail delivery system.

Issue ticket number and link

Checklist before requesting a review (magic string, indentation, comment/documentation...)

  • I have detailed the tests to do in my feature/fix in order to prevent consequents regressions (must specify in Checklist tests)
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (API Doc etc...) - (must specify in Description for target version)
  • If it is a consequent feature, I have added thorough tests.
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been added to this project (must specify in Description)

* fix(Dockerfile): #OIC-13 Updating to node JS 18.13.0

* fix(NodeJs): #OIC-14 Using crypto package to generate uuid

fix(NodeJs): #OIC-14 Add email address verification when a user has been created

* fix(Mongoose): #OIC-15 Remove default mock and remove option has benn deprecated of the Mongoose 6 release

* fix(Mongoose): #OIC-15 deprecated value for keepAlive

* feat(Nodemailer): #OIC-17 Replacement of mailgun-js library by nodemailer to send emails

* feat(Nodemailer): #OIC-17 Add views templates
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

Successfully merging this pull request may close these issues.

1 participant