Skip to content

Projeto labex#36

Open
ElisonMoises wants to merge 8 commits intomasterfrom
projeto-labex
Open

Projeto labex#36
ElisonMoises wants to merge 8 commits intomasterfrom
projeto-labex

Conversation

@ElisonMoises
Copy link
Copy Markdown
Collaborator

@ElisonMoises ElisonMoises commented Jul 11, 2022

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

INSIRA O NOME NOME DO PROJETO AQUI

O que funciona

  • DESCREVA O QUE FUNCIONA NO SEU PROJETO

O que não funciona

  • DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

Link Surge

http://scattered-pot.surge.sh/

Imagens

Labex

labenu-bot
labenu-bot previously approved these changes Jul 21, 2022
Copy link
Copy Markdown

@labenu-bot labenu-bot left a comment

Choose a reason for hiding this comment

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

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Ao entrar no site, o usuário deve ser capaz de escolher se quer fazer login de administrador ou se quer conhecer as viagens
Ao seguir no fluxo não autenticado, o usuário deve ser capaz de ver uma lista de viagens
O usuário deve ser capaz de visualizar um formulário de inscrição com um dropdown possuindo todas as viagens disponíveis
Os campos do formulário de inscrição devem possuir as validações pedidas, e caso alguma das informações não seja preenchida ou não siga as regras, a requisição não é realizada e um erro claro é mostrado na tela
Ao tentar fazer login com informações incorretas, uma mensagem de erro clara aparece para o usuário
Ao tentar fazer login com informações corretas, o usuário deve ser levado para o dashboard de administrador
Ao tentar acessar uma tela do fluxo de admin sem estar logado, o usuário deve ser redirecionado para a tela de login
O administrador deve ser capaz de adicionar uma nova viagem
Os campos do formulário de criar viagens devem possuir as validações pedidas, e caso alguma das informações não seja preenchida ou não siga as regras, a requisição não é realizada e um erro claro é mostrado na tela
O administrador deve ser capaz de visualizar os detalhes de uma viagem
Na tela de detalhes da viagem, o administrador deve ser capaz de visualizar duas listas: uma com os candidatos pendentes e uma com candidatos aprovados para a viagem
Os candidatos cadastrados no fluxo não autenticado aparecem para serem aprovados na tela de detalhe da viagem
Na tela de detalhes da viagem, o administrador deve ser capaz de aprovar ou reprovar candidatos
Ao aprovar um candidato, ele desaparece da lista de pendentes e aparece na lista de aprovados sem necessidade de atualizar a página
Ao reprovar um candidato, ele desaparece da lista de pendentes sem necessidade de atualizar a página
Quão fácil foi achar as informações no site? Fácil
O que você achou do design do site? Bom
[Desafio] Todas as requisições possuem loadings -
[Desafio] Possui funcionalidade de logout -
[Desafio] Possui boa responsividade para mobile -
Funcionalidade extra -
Feedback do código Feito
Criou as rotas corretamente e as isolou em um componente
As mensagens de erro são captadas corretamente e mostradas para o usuário, ex: alert(err.response.data.message)
Extraiu a base url para um arquivo separado e a utilizou em todas as requisições
Evitou misturar tipos de estilização (useStyles vs. styled components)
Não utilizou o método reload() para atualizar a página após a finalização de uma requisição
Utilizou os custom hooks fornecidos em aula (useForms, useRequestData)
Criou link do surge
Criou um bom readme com informações relevantes para o projeto

Comentários da pessoa avaliadora

Olá Elison!

Parabéns pela entrega do projeto, ficou muito bom!

Alguns pontos sobre seu código!

  • Você conseguiu implementar todas as funcionalidades exigidas no projeto, parabéns, faltou apenas a parte de responsividade que em algumas páginas ficou sobressaindo nas telas, para uma melhor adaptabilidade do código recomendo a utilização do @media screen, fora isso, tudo certo :)

  • Na tela de admin, ao mostrar as viagens, seria interessante colocar um texto no botão para deletar, pode acontecer de o usuário clicar lá sem saber qual a funcionalidade dele e deletar mesmo sem querer, então para uma melhor usabilidade é sempre bom deixar tudo bem objetivo, até mesmo para uma melhor acessibilidade de seu site.

  • No seu Readme você poderia trazer mais informações sobre todo o projeto como tecnologias utilizadas.

  • O botão de logout não está funcionando, você poderia ter feito uma lógica para limpar o localStorage e mandar o usuário para a tela de login.

No mais, parabéns pela entrega, mandou bem!

Tenha uma ótima semana!

Análise realizada por: Marcelo Maia

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.

3 participants