Skip to content

projeto-completo#70

Open
Marcos-vvc wants to merge 2 commits intomasterfrom
rodada-case-semana4
Open

projeto-completo#70
Marcos-vvc wants to merge 2 commits intomasterfrom
rodada-case-semana4

Conversation

@Marcos-vvc
Copy link
Collaborator

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

Amaro

Teste Case: https://github.com/amaroteam/back-end-challenge

Heroku

Documentação

Para rodar o projeto

1 - De npm install: Para instalação das dependências
2 - Rode npm run migrations: Para fazer a criação e população no banco

Ps: Na criação do produto as tags precisam ser colocadas dentro de um array.

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
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

Fala Marcos, tudo bem?

Parabéns pela entrega do seu último case! 📝🗂️🧾👏🏻🤩
Projeto redondinho, funcionando tudo que foi pedido!

Só faltou mesmo o ednpoint de pegar a lista completa, mas tudo que você implementou funciona demais :D

Muito sucesso em sua jornada meu querido!!!
Parabéns 🤘

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