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

style(account): substitui com email estilizado para receber código de login #112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ByIvo
Copy link
Member

@ByIvo ByIvo commented Dec 11, 2024

Obs: Claramente o sistema está seguindo uma regra de organização, mas eu não entendi onde eu devo colocar o template, nem como recuperar ele.

Essa primeira versão vai com o código inline, podemos discutir por aqui qual seria o local ideal.

Resultado:
image

Copy link

nx-cloud bot commented Dec 11, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit bd20b90. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@guiseek
Copy link
Member

guiseek commented Dec 15, 2024

Obs: Claramente o sistema está seguindo uma regra de organização, mas eu não entendi onde eu devo colocar o template, nem como recuperar ele.

Essa primeira versão vai com o código inline, podemos discutir por aqui qual seria o local ideal.

Resultado: image

Atualmente não existe um template de fato, por hora este código se encontra aqui

@guiseek
Copy link
Member

guiseek commented Dec 15, 2024

@ByIvo tema light não ficaria melhor?

@guiseek
Copy link
Member

guiseek commented Dec 15, 2024

Obs: Claramente o sistema está seguindo uma regra de organização, mas eu não entendi onde eu devo colocar o template, nem como recuperar ele.

Essa primeira versão vai com o código inline, podemos discutir por aqui qual seria o local ideal.

Resultado: image

Faz rebase da branch main que terá disponível uma função chamada render, o primeiro parâmetro deve ser o nome do arquivo do template, relativo ao diretório /apps/server/src/assets/templates/, tem um exemplo lá, o segundo parâmetro é um objeto que será usado na substituição das variáveis no template

Exemplo:

apps/server/src/assets/templates/user-code.html

<div>
  <h2>{{value}}</h2>
</div>

packages/account/domain/src/server/use-cases/send-user-code.ts

import { render } from '@devmx/shared-util-data/server';
    
    //...
    
    const mail = createMail(
      user.contact.email,
      render('user-code.html', code)
      `Código de autenticação`
    );

obrigado pela contribuição

@guiseek
Copy link
Member

guiseek commented Dec 15, 2024

@ByIvo sinto muito por não termos feito merge ainda, da forma que está, mas vou explicar o motivo.

Digamos que um dos ícones das redes sociais ali esteja errado e percebemos no futuro, foi feito merge usando ttps://, faltando o h no começo. Uma correção extremamente simples de resolver, oncorda? Uma good first issue é criada e veio no junto no pacote uma oportunidade pra que nenhum usuário consiga autenticar na plataforma, pq todo aquele template está ao lado das regras de autenticação, colocando uma pessoa iniciante pra alterar um dos arquivos mais importantes pro funcionamento da plataforma, entende?

Esse é um dos casos de uso que tarefas assim deveriam passar longe, justamente pra evitar riscos...

Então só ajustarmos essa separação, coisa rápida e seguimos eem frente, bele?

Agora, uma dúvida que me surgiu enquanto explicava esse contexto, como esse email ficou na barra de notificação do celular quando o e-mail chega? o código fica visível já na notificação ou a pessoa precisa abrir o email pra ver o código?

Obrigado novamente pela contribuição

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.

None yet

2 participants