Skip to content

Conversation

@helenarangel
Copy link
Collaborator

@helenarangel helenarangel commented Dec 11, 2020

Projeto LabeX

O que funciona

  • Acho que a maior parte dos componentes está funcionando bem.

O que não funciona

  • Não consegui fazer os detalhes da viagem e dos candidatos na página do administrador. Algumas outras coisas, como resetar os campos depois de preenchidos, também não consegui fazer. Tentei usar aquela propriedade timeout, mas não deu certo.

Link do Surge

satisfying-cable.surge.sh

Copy link

@leticia-chijo leticia-chijo left a comment

Choose a reason for hiding this comment

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

Oi Helena! A sua entrega do LabeX foi considerada dentro do esperado, parabéns! 🎉

Ficaram faltando algumas coisinhas mas a parte que você fez ficou bem legal. A descrição do seu PR está muito boa, além de você ter deployado o site no surge, o que é sempre ótimo. O código está organizado. Algumas coisas que deixariam seu projeto ainda melhor são:

  • Toda vez que eu atualizo a página ou volto para a rota inicial, mesmo o meu token estando guardado no local storage, eu preciso realizar o login novamente
    A requisição de aprovar ou não os candidatos parece que está funcionando, só falta dar algum feedback visual para o usuário (por exemplo, atualizar a lista e mostrar um alert)
  • Seus formulários não possuem validação, o que permite a entrada de dados incoerentes que causam erro na requisição. Além disso, não há feedback desse erro para o usuário.
  • Seria legal colocar loadings na sua aplicação que indiquem que a requisição está em andamento! Senão o usuário pode achar que o botão não funcionou e ficar apertando de novo, o que vai deixar tudo mais lento.

Acho que é isso! Se ficou alguma dúvida, pode mandar lá no canal. Em breve irei mandar um documentozinho com alguns erros comuns e suas soluções que pode te ajudar 😄

@helenarangel
Copy link
Collaborator Author

helenarangel commented Jan 6, 2021 via email

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