Skip to content

Labe x#32

Open
DouglasRPaula wants to merge 2 commits intomasterfrom
labeX
Open

Labe x#32
DouglasRPaula wants to merge 2 commits intomasterfrom
labeX

Conversation

@DouglasRPaula
Copy link
Collaborator

Projeto labeX

Como acessar a área administrativa :

email: douglasrodriguesdepaula@hotmail.com
senha: 123456

Link Surge

fortunate-oil.surge.sh

Imagens

image

image

image

Copy link
Contributor

@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? Excelente
[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

Parabéns pela a entrega do projeto! 🚀🤩

Onde podemos melhorar :

  • Quando eu tentei entrar na página com o fluxo de admin sem estar logado, não fui direcionada para a página de login

  • Ao criar viagem, estou conseguindo cadastrar uma viagem no passado 🤔

  • Aprimorar um pouquinho mais a responsividade

  • Quando temos muitas requisições, é interessante separá-las em um arquivo à parte

  • Quando temos algumas variáveis que se repetem no código, é interessante criar uma constante e chamá-la em cada utilização (no caso da URL base, seria bem legal)

  • Seria mais interessante deixar o App.js mais limpo, separando as rotas das requisições

  • Utilize mais os hooks (vai facilitar a sua vida!)

É isso, continuem com essa garra e dedicação! 💪🚀

Análise realizada por: Fayra Miranda

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.

2 participants