PR de Correção#7
Conversation
…eteFriendship e getFriendships
Feat: se aplicado, este commit adicionara endpoints createUser create…
Refactor:se aplicado, este commit corrige nome modal freind
Refactor: se aplicado, este commit corrige bug no request
Refactor: se aplicado, este commit add função idGenerator
Refactor: se aplicado, este commit corrige bug do postDatabase
Asoaresdev
left a comment
Asoaresdev
left a comment
There was a problem hiding this comment.
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, Byron.
Seu projeto está muito bom. Gostei como usou o authorization do headers para "estar logado" e as ações serem feitas daquele usuário "logado".
Testei seus endpoints, forcei erros e tudo funcionou dentro do esperado. Não estava no escopo do projeto, porém senti falta de um endpoint para pegar todos os usuários cadastrados, tive que ler o código para saber em qual endpoint meu id de usuário estaria exposto para fazer amizade com usuário que criei.
Tem uma função findFriendship = () => {}; dentro do arquivo FriendshipBusiness.ts que poderia estar fora da entrega final.
Seu código ficou organizado e bem escrito.
Parabéns pela entrega e pelo empenho. Continue assim 💪
2 participants
PR de Correção - favor não dar merge