Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature/fix and update mailer #47

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

rafael-g-depaulo
Copy link
Contributor

@rafael-g-depaulo rafael-g-depaulo commented Dec 3, 2022

Troca envio direto pelo gmail com o sendinblue, atualmente usando um plano gratuito (300 emails/dia).

importante validar com diretoria se é necessário fazer upgrade para um plano pago

Exemplo de emails de confirmação e rejeição de inscrição no curso funcionando com as mudanças do PR
image

image

@netlify
Copy link

netlify bot commented Dec 3, 2022

Deploy Preview for labfaz-prod ready!

Name Link
🔨 Latest commit 65d640c
🔍 Latest deploy log https://app.netlify.com/sites/labfaz-prod/deploys/6391d8ecbf27de000d4c5394
😎 Deploy Preview https://deploy-preview-47--labfaz-prod.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@rafael-g-depaulo
Copy link
Contributor Author

Na última atualização eu tive que consertar um problema na lib usada para conectar com o sendinblue.

A mudança (que pode ser vista em patches/nodemailer-sendinblue-transport+2.0.1.patch foi de remover o "node:" nos requires que o arquivo faz.

Usando a lib postinstall-postinstall para aplicar o patch ao final de todo yarn install que rolar (e yarn build:libs tbm para melhorar a DX), garantimos que tanto nos ambientes locais de dev quanto nos de produção o patch vai sempre ser aplicado corretamente ao arquivo node_modules/nodemailer-sendinblue-transport/lib/transport.js

@rafael-g-depaulo rafael-g-depaulo force-pushed the feature/fix-and-update-mailer branch from 945dc29 to 65d640c Compare December 8, 2022 12:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant