Feat/integration signin signup
🔗 Task no ClickUp
ID da Task: 86b6e2uty e 86b6e2uy6
📝 Descrição das mudanças
Esse MR implementa PARCIALMENTE a integração do fluxo de login e de cadastro, junto com os ajustes do Provider do Usuário e da Navbar. Digo parcialmente, pois uma das telas (cadastro de usuário) não estava devidamente implementada. Fui ver se tinha um comentário ou explicação no MR da task (!34 (merged)) mas acho que ela nem foi corrigida. Alguns pontos que identifiquei:
- A lógica de verificação de input não é validada da maneira certa (Previamente determinada com ZOD)!
- Atributos em português e totalmente diferentes do que o backend espera.
- Não possuí reações importantes a falha de cadastro (senha fraca, validação do formato certo de cpf, telefone).
- Comentários jogados e sem sentidos.
- Falta de clareza nos métodos e funções (SignUp é diferente de Login, a função de cadastro deve ser diferente da de login)
- Falta de conexão entre as duas etapas do Formulário (PersonalForm não passa as informações AcessForm)
Fiquei quase 6 horas para integrar uma função e não finalizei, sugiro reabrirem o MR da task da tela cadastro.
Outra sugestão é de que o a mesma pessoa que faça a tela, faça também a integração. Principalmente porque às vezes a tela é feita de maneira muito diferente do que o backend espera, e a pessoa que integra tem muito retrabalho.
🎯 Tipo de Mudança
- Bug fix (correção de bug)
- New feature (nova funcionalidade)
- Documentation (documentação)
- Refactoring (refatoração)
- Test (testes)
📸 Evidências
Segue abaixo algumas evidências da task de cadastro que esta na develop:
Como falei anteriormente, alguns erros de formatação de CPF e PHONE fizeram o back retornar erros peculiares (erro 422):
Código e arquivos não possuem diferenciação clara entre SignIn e SignUp, e as variáveis em português e inglês:
Validação feita no componente:
✅ Checklist
- Código segue os padrões do projeto
- Self-review foi feito
- Código foi testado localmente
📌 Notas adicionais
- Reforço a importância de corrigir o PR para e seguir boas práticas para facilitar o trabalho e entendimento do resto da equipe.
- Abri o MR para deixar registrado a minha tentativa. Se alguém quiser assumir o resto dessa task, eu ficaria agradecido!