Open luizpbello opened 1 year ago
@JohnnyKamigashima , agora só refatorar a task com os requisitos técnicos. Ai depois repassa como responsável pro @zThiago15 .
Não consegui editar o texto, não tenho autorização.
@zThiago15 acho que é isso.
Boa noite @jeffbcampos , @zThiago15 eu dei uma olhada na repo, o server fica hosteado na porta 3000 ou 3003? No código vi que foram criadas classes, mas não achei as validações nem os response codes declarados. A versão que está na repo já está finalizada?
Está na porta 3000. Realmente deixei passar algumas validações. Ja conversei com o @zThiago15 sobre as validações para fazermos por esses dias. Cheguei subir o servidor tbm no Render. Acho melhor recolocar o card em andamento pra terminar essas validações faltantes. Precisava do email da tech rate pra poder criar uma conta no Planet Scale, na criação do banco de dados.
@luizpbello temos algum email oficial para passar para o Jeff criar a conta?
@JohnnyKamigashima @jeffbcampos @luizpbello o e-mail e a senha estão aqui: https://www.notion.so/Links-uteis-95f7435b33b9431597c8739290c5f167
Jeff, o código de segurança está vindo para o meu nº de celular, pode me chamar no whatsapp se precisar :)
@JohnnyKamigashima a rota de criação de usuário está criada, o servidor está hosteado na porta 3000. No controller está os response.status 201 para a criação feita com sucesso e um status code 409 caso o usuário já exista. Caso ocorra algum outro erro retorna o status 400. Todas com uma mensagem json tbm. Tomei a liberdade de hospedar a api no render neste endereço https://api-take-a-peek.onrender.com onde pode ser feito um post com inmsonia para testar a rota createUser. Os dados estão chegando no planet scale normamente em meus testes por aqui.
@jeffbcampos não consegui rodar o server, parece que vários arquivos estão com conflitos não resolvidos no código, poderia revisar por favor?
@JohnnyKamigashima eu vi hoje mais cedo. Foram algumas alterações do commit que deram isso. Corrigi localmente mas não consegui dar o push na development. Amanhã de manhã faço um push numa nova branch para que o @luizpbello depois faça o merge. Te aviso por aqui assim que eu subir as correções.
Obrigado
Em Ter, 12 de set de 2023 22:51, Jeferson Vieira Borges de Campos < @.***> escreveu:
@JohnnyKamigashima https://github.com/JohnnyKamigashima eu vi hoje mais cedo. Foram algumas alterações do commit que deram isso. Corrigi localmente mas não consegui dar o push na development. Amanhã de manhã faço um push numa nova branch para que o @luizpbello https://github.com/luizpbello depois faça o merge. Te aviso por aqui assim que eu subir as correções.
— Reply to this email directly, view it on GitHub https://github.com/TechRate-Lab/TR-Backend/issues/39#issuecomment-1716813883, or unsubscribe https://github.com/notifications/unsubscribe-auth/AL62ERMCFYRXS3PM4JTVMBDX2EGQ7ANCNFSM6AAAAAAZHGBKKM . You are receiving this because you were mentioned.Message ID: @.***>
@JohnnyKamigashima e @jeffbcampos já aprovei lá e fiz o merge. Mas não testei aqui
@luizpbello vlw luiz. Fiz um git clone aqui e agora rodou na minha máquina. Acredito que o @JohnnyKamigashima tbm dará certo.
Fiz mais um teste e mesmo com uma senha mais complexa, ele retorna Senha fraca, você sabe por que? @jeffbcampos
@JohnnyKamigashima o erro tava no regex, ja ajustei aqui localmente. Quando for feito o push aviso aqui.
Ok
Criar uma api para cadastro de usuários gravando os dados recebidos em json utilizando método com criptografia que não deixe os dados visíveis ao usuário.