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

integração - registro #38

Open
Tracked by #57
viniciuscosmome opened this issue Sep 6, 2023 · 30 comments
Open
Tracked by #57

integração - registro #38

viniciuscosmome opened this issue Sep 6, 2023 · 30 comments
Assignees

Comments

@viniciuscosmome
Copy link
Contributor

No description provided.

@dayanecol
Copy link
Contributor

  • Instalar o Axios npm install axios;
  • No componente: SignUpForm > index.tsx
  1. Importar o axios
import axios from 'axios';
  1. Colocar a URL base em uma variável
const URL = ' https://routinely-api-dev.onrender.com'; //Essa é um boa prática pois facilita na identificação da API e caso precise alterar
  1. No submitSignUpForm:
const submitSignUpForm = (data: ISignUpInput, event: React.FormEvent) => {
	event.preventDefault();
    	// criar o body 
  const body = {
    name,
    email,
    password,
    acceptedTerms: termsOfUse
  };
	try {
    const response = await axios.post(URL+"/auth/register", body);
    
    console.log('Resposta da API:', response.data);

  } catch (error) {
  }
  };

@huilianni
Copy link

Aguardando o teste da @lilianekassab e @carol-schmidt .

@carol-schmidt
Copy link

Falei com a Dai no inicio da semana e estou aguardando retorno de quando posso testar novamente.

@hartmannjean
Copy link

@dayanecol vai testar e retornar para @carol-schmidt

@Soaressluiss
Copy link
Collaborator

Pronto para testes novamente (Dayane corrigiu o código).
@carol-schmidt

@carol-schmidt
Copy link

@dayanecol @Soaressluiss o que deveria acontecer após a criação da conta? Ele fez duas chamadas, mas não me mostrou se deu sucesso ou o que ocorreu.

Image

@carol-schmidt
Copy link

@veronicasvalerio conseguiria por a US dessa task, por favor

@dayanecol
Copy link
Contributor

Na integração web frontend faltam os casos de erros, por exemplo, e-mail já cadastrado.
Na API faltam os casos que não são de sucesso, para utilizarmos como referência.

@huilianni
Copy link

huilianni commented Oct 19, 2023

Favor verificar o item acima @Soaressluiss .

@huilianni huilianni assigned Soaressluiss and unassigned marinsgui Oct 19, 2023
@Soaressluiss
Copy link
Collaborator

Não tenho certeza (me corrijam se estiver errado), mas essa parte de erros vem pela API no caso com o pessoal do Back-End, então é essa parte fica com eles adicionar.

@veronicasvalerio
Copy link

@veronicasvalerio conseguiria por a US dessa task, por favor

Vou verificar com a Manu @carol-schmidt

@Soaressluiss
Copy link
Collaborator

  • Mensagens de erro padronizadas;
  • layout dos inputs alterados de acordo com style-guide do figma;

Disponível para testes novamente, @carol-schmidt @lilianekassab

@lilianekassab
Copy link

Mensagens padronizadas.ok
layout de acordo com o figma. ok

Integração Registro.docx

OBS:
Falta a mensagem no campo email, quando o mesmo já está sendo utilizado.
"Este e-mail já está sendo utilizado"

@carol-schmidt
Copy link

carol-schmidt commented Nov 14, 2023

@Soaressluiss @staviasz estava fazendo meu login, e acabei saindo e ir tentar fazer um novo cadastro e ficou assim esse campo de e-mail, sendo o campo do login está limpo,não estava mais preenchido.

Image

@carol-schmidt
Copy link

@Soaressluiss @staviasz aqui não estamos informando o usuário de nada, tentei fazer o cadastro com o mesmo usuário da minha conta, faltou essa informação.

Image

@Soaressluiss
Copy link
Collaborator

Image

Bugs corrigidos, disponível para testes novamente! @carol-schmidt @lilianekassab

@carol-schmidt
Copy link

carol-schmidt commented Nov 22, 2023

@Soaressluiss @staviasz após o preenchimento ainda está com esse fundo branco os campos de nome e e-mail, mas não deveria, deveria ficar sem para não tapar a parte azul.

Image

@Soaressluiss
Copy link
Collaborator

Soaressluiss commented Nov 22, 2023

Ah, sim! Desculpa não ter avisado... Por ser uma mudança simples só de estilização, o @staviasz fez na branch que ele está trabalhando, achei que ele enviaria essa branch antes de vocês QAs pegarem para testar, então podem dá como feita.

Quando fizermos o merge para a develop, ele irá entrar automaticamente.

@staviasz
Copy link
Contributor

Já subi a correção @Soaressluiss @carol-schmidt

@carol-schmidt
Copy link

@Soaressluiss @staviasz

Aqui ajustei a senha de baixo primeiro e depois a de cima e ficou ainda aparecendo a mensagem que elas não estavam iguais.

Image

@carol-schmidt
Copy link

carol-schmidt commented Nov 29, 2023

@Soaressluiss @staviasz

O cadastro está sendo possivel realizar somente com o nome, não está sendo necessário colocar um sobrenome, acredito que isso não deveria ser possivel.

@staviasz
Copy link
Contributor

staviasz commented Nov 29, 2023

@carol-schmidt não foi passado se deveria conter o sobrenome, mas se for necessario, podemos implementar uma logica para validar se há nome e sobrenome ou colocar um campo especifico para o sobrenome

@carol-schmidt
Copy link

@staviasz pois é, mesmo fui olhas a US, acho que na minha visão por ser um cadastro deveria ter nome e sobrenome, mas acho válido levar para a reunião hoje, só para ver o que o pessoal acha, pode ser?

@staviasz
Copy link
Contributor

@carol-schmidt podemos levar para reunião e se aprovado certamente pode ser implementado

@staviasz
Copy link
Contributor

staviasz commented Dec 2, 2023

@carol-schmidt já fiz a correção dos campos de senha, verifica pfv

@carol-schmidt
Copy link

@staviasz temos como não deixar essa borda vermelha, pensando que as senhas já estão iguais?

WhatsApp.Video.2023-12-04.at.14.34.14.mp4

@staviasz
Copy link
Contributor

staviasz commented Dec 4, 2023

@carol-schmidt teoricamente isso não era para estar acontecendo, vou dar uma olhada

@staviasz
Copy link
Contributor

staviasz commented Dec 4, 2023

@carol-schmidt verifica se está tudo ok, por favor

@carol-schmidt
Copy link

@staviasz fica aparecendo somente a mensagem ainda.

Image

@staviasz
Copy link
Contributor

staviasz commented Dec 4, 2023

@carol-schmidt verifica se esta tudo certo pfv

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

No branches or pull requests

10 participants