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 - login #39

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

integração - login #39

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

Comments

@viniciuscosmome
Copy link
Contributor

No description provided.

@dayanecol
Copy link
Contributor

dayanecol commented Sep 19, 2023

  • No componente: SignInPage > 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 handleSubmitSignIn:
async function handleSubmitSignIn (data: ISignUpInput, event: React.FormEvent) {
	event.preventDefault();
    	// criar o body 
  const body ={
        email: data.email,
        password: data.password,
        remember: data.rememberPassword,
      }};
	try {
    const response = await axios.post(URL+"/auth", body);
    
    console.log('Resposta da API:', response.data);

  } catch (error) {
  }
  };

@hartmannjean
Copy link

hartmannjean commented Oct 11, 2023

Fazer reunião para tirar dúvidas, com @Soaressluiss e @dayanecol, @hartmannjean e @OdisBy

@dayanecol
Copy link
Contributor

dayanecol commented Oct 18, 2023

Fiz uma reunião com o @Soaressluiss ele me apresentou o problema que teve com persistência, mas não consegui testar na minha máquina e investigar o problema.

@Soaressluiss
Copy link
Collaborator

Após uma conversa esclarecedora com o Cosmo sobre o bug, testarem o código amanhã ou outro dia o rápido possível.

@Soaressluiss
Copy link
Collaborator

Pronto para testes. @carol-schmidt @lilianekassab

@carol-schmidt
Copy link

@Soaressluiss @dayanecol sobre a mensagem de erro quando o usuário coloca a senha ou e-mail errado, deveria estar vindo em um formato diferente, até fui falar com as meninas de UX e elas vão colocar no figma.
Mas enquanto isso vou colocar o print de como está e de como deveria ser.

Como esta:

Image

Como deveria ser:

Image

Deve aparecer a mensagem em baixo do campo que está trazendo a informação errada.

@Soaressluiss
Copy link
Collaborator

@carol-schmidt

Sugiro que essa mudança seja repensar e debatida em reunião.

Esse padrão foi criado e apresentando para as PMs e UXs que estavam presentes e validado nas primeiras sprints de Layouts.

A API não possui distinção de qual campo de input vem o erro. (Pelo que verifiquei até o momento). Posso estar enganado.

Altera todas as páginas (praticamente todas há campos de Inputs) irá gerar um retrabalho e além de que teremos que trocar toda a lógica por trás em cada página, atrasada ainda mais o projeto.

@carol-schmidt
Copy link

@RoutinelyOrganization/ux-ui conseguem dar uma olhada no retorno Luis sobre os inputs.

@izabellaurias
Copy link

Oi gente, mas não foi feito como o padrão estabelecido no style guide desde a primeira vez, e é esse tipo de coisa que eu peço para vocês avisarem a gente. Se dá mais trabalho ou se a API não identifica o campo exato para retornar a gente precisa saber para alterar pra gente também.

@carol-schmidt
Copy link

@veronicasvalerio @dedeski conseguem nos dar uma ajuda em relação a essa questão a cima

@dedeski
Copy link

dedeski commented Oct 31, 2023 via email

@Soaressluiss
Copy link
Collaborator

Ajustes realizados (inputs e mensagens de erros padronizados) em loginPage.
Disponível para testes novamente @carol-schmidt.

@carol-schmidt
Copy link

@Soaressluiss e @staviasz o campo de e-mail ao preencher ele e sair o nome do campo "e-mail' está sobrescrevendo o e-mail que foi digitado

Image

@carol-schmidt
Copy link

@Soaressluiss @staviasz esse checkbox de lembrar meu acesso seria para o que? Não seria para a pessoa não precisar digitar o usuário e senha no próximo acesso? Se for isso não está rolando. Deixei marcado e mesmo assim no próximo acesso precisei digitar novamente.

Image

@carol-schmidt
Copy link

carol-schmidt commented Nov 14, 2023

A palavra "e-mail" poderia ficar sem esse fundo branco, para não cobrir o azul

Image

@Soaressluiss
Copy link
Collaborator

Image

  • Label do campo de email arrumado, @carol-schmidt disponivel para testes!

@Soaressluiss
Copy link
Collaborator

Image

Fiz a verificação em 5 navegadores diferentes (Chrome, Opera, Brave, Edge e Mozilla) em ambos o checkbox está funcionando da maneira esperada (quando marcado, o usuário não precisou fazer login novamente).
@carol-schmidt

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

7 participants