-
Notifications
You must be signed in to change notification settings - Fork 5
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
Conversation
Co-authored-by: Flávio Viera Leão <flavio.vl@gmail.com>
There was a problem hiding this 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!!
@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. |
There was a problem hiding this 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
@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. |
There was a problem hiding this 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.
There was a problem hiding this 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.
@shayanealcantara foi removido o link para senha perdida, todos os requisitos do PR foram atendidos. |
Descrição
Foi implementada a autenticação de usuário (login)
Resolve (Issues)
Issue #44
PRs relacionados
Tarefas gerais realizadas
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)
Ao ser autenticado com sucesso você será redirecionado para a home e poderá ver os tokens registrados (que estão guardados nos cookies)