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

#44 Login implementado #6

Merged
merged 10 commits into from
Oct 2, 2019
Merged

#44 Login implementado #6

merged 10 commits into from
Oct 2, 2019

Conversation

RenatoBrittoAraujo
Copy link
Collaborator

@RenatoBrittoAraujo RenatoBrittoAraujo commented Sep 27, 2019

Descrição

Foi implementada a autenticação de usuário (login)

Resolve (Issues)

Issue #44

PRs relacionados

Branch PR
feature/44-user-authentication (backend) link

Tarefas gerais realizadas

  • Criar sessão por meio da api;
  • Criar formulário no frontend;
  • Comunicação de token entre front e back;
  • Envio do formulário do frontend para o backend;
  • Resposta de sucesso ou fracasso enviado do backend para o frontend;
  • Frontend deve processar a resposta do backend, informando o usuário
  • Adicionar botão de login;
  • Adicionar botão de logout;
  • Encerrar sessão por meio da api;

Para testar:

Execute o backend (branch: feature/44-user-authentication) e frontend (branch: feature/44-user-signin) pelo docker-compose usando suas portas padrões, crie um usuário diretamente pelo backend por esse caminho e depois entre na página '/signin' do frontend e faça a autenticação de usuário usando email e senha. Você será direcionado para a home onde você pode ver os tokens access e refresh e fazer logout ao pressionar o botão.

Página de signin (mobile)
image

Ao ser autenticado com sucesso você será redirecionado para a home e poderá ver os tokens registrados (que estão guardados nos cookies)

image

Copy link
Collaborator

@vitorcx vitorcx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Só um detalhe, os textos, tanto na interface quanto nas rotas devem estar em português, acredito que seja tranquilo corrigir isso. Tirando esse detalhe, ficou muito bom, parabéns!!

@RenatoBrittoAraujo
Copy link
Collaborator Author

@shayanealcantara @vitorcx A página de signin foi traduzida para português e mais validações de input foram implementadas, completando todos os requisitos do PR.

Copy link
Collaborator

@shayanealcantara shayanealcantara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RenatoBrittoAraujo @flaviovl Não estou conseguindo reproduzir a tela que aparece o token. Está dando erro de email ou senha, apesar de eu conferir mil vezes. Nesse link mostro o último processo que fiz.
https://drive.google.com/file/d/1UE7XKzhMA8YLfQcI_bKwAeAe2ca56aQg/view

@RenatoBrittoAraujo
Copy link
Collaborator Author

RenatoBrittoAraujo commented Oct 1, 2019

@shayanealcantara O host que você está usando para o front-end é 0.0.0.0:8000, acredito eu que se você abrir o console de desenvolvedor você encontrará um erro ao apertar o botão de signin informando sobre "falta de CORS Headers". Os hosts que eu adicionei ao backend foram localhost:8080 e localhost:8000. Vou adicionar o host 0.0.0.0:8080 no back-end para você poder utilizar o front-end na configuração que você está usando. Aparentemente tudo está certo com o front-end.

Edit: Adicionado no back-end o host 0.0.0.0:8000
image

Copy link
Collaborator

@shayanealcantara shayanealcantara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Executei com as novas alterações no backend e consegui me autenticar. Bom trabalho.

Copy link
Member

@fabiolamfleury fabiolamfleury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peço que vocês deixem comentado o botão "Esqueceu sua senha" pois botões de funcionalidades que ainda não existem também não devem existir.

@RenatoBrittoAraujo
Copy link
Collaborator Author

@shayanealcantara foi removido o link para senha perdida, todos os requisitos do PR foram atendidos.

@shayanealcantara shayanealcantara merged commit bab50b0 into develop Oct 2, 2019
@shayanealcantara shayanealcantara deleted the feature/44-user-signin branch October 2, 2019 13:20
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

Successfully merging this pull request may close these issues.

5 participants