feat: Email Service (Template Preview + SES template uploader)#419
feat: Email Service (Template Preview + SES template uploader)#419
Conversation
@Joabesv
ps: ps2: |
|
Cara por pontos @grecoww 2 vite consegue sim renderizar html para o seu pc local usando o próprio rollup options (não q a gente deva utilizar), mas o meu ponto principal é o seguinte, pq não podemos rodar vite dev e ver o código vue em alto nível no browser? Não tem problema q ele esteja plugado no framework pra mostrar isso
Último ponto, o package pode se chamar somente email Fica a vontade pra complementar @mateusbrg |
|
@Joabesv Atualizarei o nome para somente: |
|
@grecoww Vc pode sempre rodar o comando build pra gerar o html e txt localmente, esse cara não deve ser comittado inclusive. |
@Joabesv Show, acho uma ótima solução, agnt pode ir uma call pra conversar e se alinhar melhor? Porque por mais que concorde com a solução ainda não entendi como ela será viável. (Não entendo onde o html+txt do template isolado pode sair na build dist no vite). Mas enquanto isso vou continuando a parte de visualizacao dos templates com o vite. |
grecoww
changed the title
|
Acompanhando aqui pessoal! Bora pra cima |
| if (error.name === 'AlreadyExistsException') { | ||
| // Se template ja existe, atualiza | ||
| const updateCommand = new UpdateTemplateCommand(templateParams); | ||
| await sesClient.send(updateCommand); | ||
| console.log(`✅ Template ${templateName} atualizado no SES`); | ||
| } else { | ||
| console.error(`Erro ao fazer upload do template ${templateName}:`, error); | ||
| } |
There was a problem hiding this comment.
possibilidade de refactor para handle error
Facilita leitura
| await sesClient.send(createCommand); | ||
| console.log(`✅ Template ${templateName} criado no SES`); | ||
| } catch (error) { | ||
| if (error.name === 'AlreadyExistsException') { |
There was a problem hiding this comment.
existe um comando mais adequado para descobrir o tipo de uma exceção!
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/instanceof
| if (fileName.endsWith('.vue')) { | ||
| const templateName = path.parse(fileName).name; | ||
| console.log(`Uploading template: ${templateName}`); | ||
| await sendTemplateToSES(templateName); | ||
| } |
There was a problem hiding this comment.
possibilidade de função para melhorar leitura
alem disso, procure entender um pouco melhor o conceito de early return (caminho das negativas) ele permite uma leitura mais fluida do codigo, aumentando manutenabilidade (quem for dar manutencao tem mais facilidade)
| <Html lang="en"> | ||
| <Text>{{ title }}</Text> | ||
| <Hr /> | ||
| <Button href="https://example.com">Click me</Button> |
There was a problem hiding this comment.
O fato de ser um teste me deixa com a duvida se deveria estar aqui.. é um arquivo que deveriamos ter apagado?
There was a problem hiding this comment.
Aqui, segui a mesma filosofia justificada na omissão do linter. Por conta de ser uma POC, deixei essa template temporariamente no PR, para que os demais membros possam testar rapidamente a funcionalidade de visualizar os templates com o servidor vite, com o roteamento primitivo que eu desenvolvi.
| }, | ||
| }, | ||
| { | ||
| ignores: ['apps/email/**/*'], |
There was a problem hiding this comment.
Por conta de ser apenas uma POC, fiz isso para acelerar o desenvolvimento. Com o tradeoff de não ter um linting me avisando de possiveis más práticas, com certeza.
|
att: estou trabalhando nisso tbm no momento |
5 participants
No description provided.