Skip to content

Conversation

@victorfdovale
Copy link
Collaborator

@victorfdovale victorfdovale commented Apr 6, 2022

Link Surge : cubo-frontend.surge.sh
Documentação API : https://documenter.getpostman.com/view/18386367/UVyvuZ8j
Deploy no Heroku: https://cubo-back-end-labenu.herokuapp.com/

Copy link

@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
Endpoint para enviar dados do usuário
Endpoint para pegar informação de todos usuários
Criou uma entidade para o usuário
O consumo da API pode ser feito via REST ou GraphQL
Endpoint para pegar informações de cada usuário -
Usou algum banco de dados
Fez testes unitários -
Criou documentação do Postman
Fez um readme com instruções de como rodar o projeto -
Seguiu o layout padrão
Quão fácil foi achar as informações no site? Médio
Há um formulário com campos obrigatórios name, lastname e e participation
Ao enviar dados através do formulário, a tabela deve ser atualizada
É possível ver uma tabela com o name, lastname e a participation dos usuários já existentes
Disponibilizou o link do deploy
Fez um readme com instruções de como rodar o projeto -
Feedback do código Feito

Comentários da pessoa avaliadora

Olá, Victor! Gostaria de primeiramente parabenizá-lo por concluir o segundo case de final de curso da Labenu! \o/

Pude perceber que você conseguiu implementar muito bem o projeto proposto, integrando corretamente o frontend ao backend! Fez um projeto FullStack totalmente funcional, parabéns! ;)

De toda forma, gostaria de pontuar algumas coisas que possam ajudar:

  1. Senti falta de uma documentação README.md, da API e/ou outros que descrevessem o projeto. Seria interessante a construção destes, pois daria mais noção sobre o funcionamento do seu fluxo de front e back, para que caso alguém queira acessar na prática o seu código, consiga ter uma clareza maior sobre o funcionamento como um todo ;)

  2. Ao rodar o seu deploy do frontend, percebi que tudo funciona e está integrado. Porém, ao tentar reiniciar o projeto, para testar a persistência dos dados, percebi que os dados dos participantes permanecem, mas o gráfico não é inicialmente carregado. :/ Isto ocorre, pois você setou como estado inicial de chartData um array vazio. Como sugestão, você poderia pensar em utilizar o customHook useRequestData para disparar inicialmente a requisição, trazendo as informações iniciais do gráfio ;)

No mais, muito sucesso na sua jornada e parabéns pela entrega! :)

Análise realizada por: Bruno

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