Skip to content

Rodada cases semana3 - #56

Open
victorfdovale wants to merge 4 commits into
masterfrom
rodada-cases-semana3
Open

Rodada cases semana3#56
victorfdovale wants to merge 4 commits into
masterfrom
rodada-cases-semana3

Conversation

@victorfdovale

Copy link
Copy Markdown
Collaborator

@labenu-bot labenu-bot left a comment

Copy link
Copy Markdown

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
Fez endpoint para inserção de dados
Os dados json cedidos pelo cliente foram enviados para banco de dados através do endpoint de inserção de dados
Fez endpoint para consulta de produtos
É possível ver uma lista com todos os produtos
É possivel fazer uma busca por um produto específico usando id -
É possivel filtrar produtos por nome ou tags e receber uma lista todos os produtos com aquela respectiva busca. -
Usou algum banco de dados
Fez testes unitários -
Criou uma entidade para produto
Criou um 'README' explicando como executar o projeto -
Criou um cache para consulta -
Criou documentação do Postman
Criou autenticação usando JWT. -
Feedback do código Feito
Separou responsabilidades usando os princícios de arquitetura de softWare
Isolou as configurações do servidor das lógicas de negócio
Isolou as entidades da aplicação
Tratou corretamente os erros

Comentários da pessoa avaliadora

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

Pude perceber que você conseguiu implementar bem algumas requisições solicitadas, além de utilizar a lógica de camadas, orientação a objetos e também uma documentação mínima para a API! :)

De toda forma, gostaria de trazer algumas sugestões que podem ajudar num próximo projeto:

  1. Pude observar que você fez uma documentação da API, o que é super bacana! :) Porém, senti que faltou informações mais descritivas de utilização e também de um README.md, com instruções sobre o funcionamento da própria API (quais variáveis utilizar, quais limitações, etc) e também uma descrição mais funcional de como reproduzir o seu backend. Pense nisso! ;)

  2. Ao analisar sua requisição de criar novos produtos, percebi que ela não permite que o usuário crie mais de uma tag por produto :/ Como sugestão, você poderia pensar em receber na requisição um array de tags, e com estes mapear os dados e inserir, um a um, no banco de dados as novas tags (em uma tabela somente para tags ou usando algum método de conversão string-array para transferir os dados do banco). Pense nisso! :)

  3. Senti falta de testes unitários no seu fluxo, visto que eles são requisitos obrigatórios do projeto :/

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.

2 participants