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

Cadastro do usuário #3

Open
victorjoaoss opened this issue Mar 21, 2023 · 15 comments · Fixed by #5
Open

Cadastro do usuário #3

victorjoaoss opened this issue Mar 21, 2023 · 15 comments · Fixed by #5

Comments

@victorjoaoss
Copy link

victorjoaoss commented Mar 21, 2023

PetJournal/petjournal.product#2

@dansenpir dansenpir self-assigned this Mar 23, 2023
@dansenpir
Copy link
Contributor

dansenpir commented Mar 23, 2023

  • O campo “Nome” não pode conter menos de 3 caracteres.

  • O campo “Nome” não pode conter números e nem caracteres especiais.

  • O campo “Sobrenome” não pode conter menos de 3 caracteres

  • O campo “Sobrenome” não pode conter números e nem caracteres especiais.

  • O campo E-mail deve ser válido, isto é seguir a estrutura de um e-mail: user@dominio.

  • O email não pode estar cadastrado na base de dados.

  • O campo telefone deve seguir a estrutura de DDD, com dois dígitos e Número com 9 dígitos numéricos.

  • O número de telefone não pode estar cadastrado na base de dados.

  • O campo Senha deve ser obrigatório.

  • O campo Senha deve conter no mínimo 8 caracteres alfanuméricos e especiais.

  • O campo “Confirmar Senha” deve ser igual ao campo "Senha".

  • Todos os campos são de preenchimento obrigatório.

  • O usuário deve receber um email de confirmação:

Assunto: Verificação de email PetJournal

Mensagem:
Olá, Nome da pessoa.
Sua conta PetJournal está quase pronta. Para ativá-la, por favor confirme o seu endereço de email clicando no link abaixo.
Ativar minha conta/Confirmar meu email
Se você não se cadastrou na PetJournal recentemente, por favor ignore este email.
Equipe PetJournal

@dansenpir
Copy link
Contributor

@victorjoaoss @Marii-brito

Seria possível quebrar o envio de email de confirmação em uma nova task?
Acredito que este item possui complexidade suficiente para estar isolado.

@dansenpir
Copy link
Contributor

@victorjoaoss @Marii-brito

Sobre o telefone: ele poderia ser obrigatório? Isto é, o usuário deve informar um telefone, mesmo que não consigamos por ora validar o número informado com código de verificação por SMS.
Acredito que faça mais sentido.

@victorjoaoss
Copy link
Author

@dansenpir só afim de verificar se estamos na mesma pagina, você está sugerindo que seja criado uma US somente para o requisito de "envio de email de confirmação"?

@victorjoaoss
Copy link
Author

Em relação ao "número de telefone obrigatório", realmente faz mais sentido ele de forma obrigatória, eu só fiz a mudança pois entendi a partir da nossa reunião, que não teria como, mais pelo visto eu me equivoquei. Vou fazer a mudança na US.

@dansenpir
Copy link
Contributor

@dansenpir só afim de verificar se estamos na mesma pagina, você está sugerindo que seja criado uma US somente para o requisito de "envio de email de confirmação"?

Pode ser apenas uma nova task, dentro da mesma US.
Não sei bem como seria organizado isso no GitHub Projects, mas se for para entregar tudo o que está nesse card, vou precisar de mais um sprint inteira no mínimo, pois acrescenta complexidade.
Porém, se o entregável for o fluxo de cadastro de usuário com as devidas validações, é bem provável que eu finalize amanhã, possibilitando uma atuação em outras US/tasks dos devs @maztt e @matheusgondra.

@dansenpir
Copy link
Contributor

Em relação ao "número de telefone obrigatório", realmente faz mais sentido ele de forma obrigatória, eu só fiz a mudança pois entendi a partir da nossa reunião, que não teria como, mais pelo visto eu me equivoquei. Vou fazer a mudança na US.

Beleza!

Vou adaptar o código a essa nova regra de negócio.

@Marii-brito
Copy link

@dansenpir , pelo o que eu entendi, a confirmação de e-mail está dentro do fluxo de cadastro do usuário.
Se não conseguimos finalizar essa US, acho melhor aguardamos a nova task de confirmação de e-mail.

@dansenpir
Copy link
Contributor

@Marii-brito, entendo que poderiam ser duas tasks dentro da US de cadastro de usuário.
Essa que realiza o cadastro e uma nova que faz a validação do cadastro por email.

@dansenpir
Copy link
Contributor

@victorjoaoss @Marii-brito
Descobri que é possível linkar mais de um Pull Request a uma Issue, que por consequência reflete nos cards.
Sendo assim, logo finalizo o cadastro de usuário e envio o Pull Request para liberar a atuação do @maztt e @matheusgondra e crio uma nova branch para dar continuidade à US com a validação do email.

@dansenpir
Copy link
Contributor

Estou com dificuldades para testar as ações com banco de dados e isso tem atrasado a entrega da task.

@Marii-brito
Copy link

Okay, @dansenpir ! Obrigada pelo retorno.

@dansenpir dansenpir linked a pull request Apr 17, 2023 that will close this issue
@matheusgondra
Copy link
Contributor

@Marii-brito vou terminar a última parte da task, o envio do email. Queria saber qual é o link que é enviado no email

Assunto: Verificação de email PetJournal

Mensagem:
Olá, Nome da pessoa.
Sua conta PetJournal está quase pronta. Para ativá-la, por favor confirme o seu endereço de email clicando no link abaixo.
Ativar minha conta/Confirmar meu email
Se você não se cadastrou na PetJournal recentemente, por favor ignore este email.
Equipe PetJournal

@Marii-brito
Copy link

Oi @matheusgondra , link que levará o usuário para a tela de Login novamente, com a msg que o email esta confirmado. Mas pelo o que eu vi não temos essa tela em protótipo pelo time de @PetJournal/ux-team. Podemos conversar sobre isso hoje na reunião ?

@rodrigofmeneses
Copy link
Member

@PetJournal/agile-team @PetJournal/product-team

Olá pessoal, surgiu algumas dúvidas a respeito dessa US.

  • O link enviado deve ter tempo de expiração?
  • O que acontece se o usuário tentar logar antes de confirmar o email?
  • O que acontece se o usuário tentar criar outra conta com o mesmo email antes de confirmar o email?
  • O que acontece se o usuário tentar recuperar a senha mesmo antes de confirmar o email?

Talvez isso tenha sido debatido no refinamento dessas tasks antes da minha chegada ao projeto, agredeço a explicação.

Essas dúvidas são pertinentes pois atualmente não estamos levando em consideração esse processo de confirmação, então teremos que fazer algumas refatorações.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Waiting QA
Development

Successfully merging a pull request may close this issue.

5 participants