Skip to content

Rodada case s3#69

Open
Marcos-vvc wants to merge 6 commits intomasterfrom
rodada-case-s3
Open

Rodada case s3#69
Marcos-vvc wants to merge 6 commits intomasterfrom
rodada-case-s3

Conversation

@Marcos-vvc
Copy link
Collaborator

@Marcos-vvc Marcos-vvc commented Aug 3, 2022

Case Cubo FullStack

Documentação

Heroku

https://casecubo.herokuapp.com/user/

Link

Site

Captura de tela 2022-08-03 110900

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? Fácil
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

Fala Marcos, tudo bem?

Parabéns pela entrega do seu terceiro case! 📝🗂️🧾👏🏻🤩
Projeto muuuito bom!
Tudo funcionando conforme esperado :)

Consegui visualizar a tabela e grafico atualizados no momento que insiro uma nova participação.
Vi que no back você criou o endpoint de deletar, seria interessante ter implementado no front :)

No mais, parabéns pelo projeto. Muito bom 🤘

Análise realizada por: Murilo Terenciani

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