Skip to content

PR de Correção #6

Open
MirandaFayra wants to merge 25 commits intocorrecao-projetofrom
master
Open

PR de Correção #6
MirandaFayra wants to merge 25 commits intocorrecao-projetofrom
master

Conversation

@MirandaFayra
Copy link

@MirandaFayra MirandaFayra commented Feb 9, 2023

PR de correção, não margear.

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
Refatorou o endpoint de cadastrar,criar e buscar posts
Criou o endpoint de fazer amizade
Criou o endpoint de desfazer amizade
Criou o endpoint de ver feed
Requisições inválidas devolvem respostas de erros claras
[Desafio] Realizou algum desafio/exercício complementar -
Feedback do código Feito
Isolou as entidades da aplicação (usuários/posts)
Utilizou corretamente as sintaxes de classe
Utilizou corretamente os routers do express
Realizou um bom README.md, com o link do deploy

Comentários da pessoa avaliadora

  • Parabéns pela entrega do projeto! 👏🏽


Onde podemos melhorar :


  • Acho que seu README.md poderia ser mais completo, com rodar localmente, como criar as tabelas, tecnologias usadas etc… ;

  • Muito bom como você organizou o seu código, dividindo em pastas de acordo com cada funcionalidade ;

  • Muito bom o tratamento de erros, com mensagens permitindo que a pessoa veja onde errou e onde precisa passar parâmetros;

  • Cuidado com as importações não utilizadas;

  • Muito bom o padrão que você está usando nos commits, continue assim ;

  • Cuidado com o padrão de idiomas escolhidos para desenvolver o seu código. Alguns tratamentos de erros estão em inglês e outros em português;

  • Cuidado com os caminhos dos seus endpoints, por exemplo no caso do pegar usuário por id, vc ao invés de usar o get poderia utilizar o nome da entidade/:id ;



Lê, é lindo demais acompanhar sua evolução e dedicação aqui dentro! Continue nessa pegada! 




Muito sucesso na caminhada! 🚀



Espero que seu sim chegue logo (se jogue, mesmo com medo) e que possamos comemorar mais uma conquista sua! 


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.

3 participants