-
Notifications
You must be signed in to change notification settings - Fork 11
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
Cliente de email integrado com SendGrid #23
Conversation
Atualização nas automatização de issues
Codecov Report
@@ Coverage Diff @@
## dev #23 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 13 +2
Lines 73 84 +11
Branches 5 5
=========================================
+ Hits 73 84 +11
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por mim está tudo ok.
@rodrigondec, pode ser o merge normal. |
Descrição das Mudanças
Adiciona uma abstração para integrar com o SendGrid.
Para iniciarmos a utilização do SendGrid, precisamos executar algumas tarefas:
Os ítens listados acima não impedem que essa PR seja mergeada.
Bugs Corrigidos
Mudanças na API
Nenhuma
Mudanças de comportamento
Nenhuma
Testing Procedure
Testes automatizados:
Testes manuais:
Acesse a API em modo terminal:
Após acessar a api em modo CLI, rode os seguintes comandos:
Você deverá ver uma mensagem semelhante a:
Verifique sua caixa de e-mail, você deverá ter recebido uma mensagem indicando que foi enviada via SendGrid:
PR Checklist
PR Checklist do time