Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inicio do Exercicio de Revisão do Modulo 4 #52

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

GabrielGCJ
Copy link
Collaborator

@GabrielGCJ GabrielGCJ commented Feb 11, 2022

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

INSIRA O NOME NOME DO PROJETO AQUI

O que funciona

  • Tudo

O que não funciona

  • Nada

Link Surge

A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI

Copy link

@ClaraMeirelles ClaraMeirelles 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
Tabela de usuário contendo id (primary key), name, email e password
Endpoint de criação de usuário funcionando
Endpoint de busca por todos os usuários funcionando
Tabela de produtos contendo id (primary key), name, price e image_url
Endpoint de criação de produto funcionando
Endpoint de busca por todos os produtos funcionando
Tabela de compras contendo id (primary key), user_id (foreign key), product_id (foreign key), quantity e total_price
Endpoint de busca por todas as compras de um determinado usuário funcionando
[Desafio] O endpoint de busca por todos os produtos é capaz de ordenar em ordem alfabética ou preço -
[Desafio] O endpoint de busca por todos os produtos é capaz de filtrar os resultados através do nome -
[Desafio] O endpoint de busca por todos os usuários também retorna as compras registradas de cada um -
Feedback do código Feito
Type para representar usuário, produto e/ou compra
Nomes adequados para variáveis e rotas -
Migration para criação de tabelas -

Comentários da pessoa avaliadora

Olá Gabriel!

Parabéns por mais uma semana de back end e mais um projeto!

Sobre o teu código, está bastante ordenado e limpo, ainda dá para melhorar o código, mas é visível a sua evolução no backend, parabéns!

Pontos de atenção:

  • Você criou um type pra user mas não chegou a usar ele na hora de receber um novo usuário ou enviar um usuário na requisição... é muito bom poder tipar as entradas e saídas, para deixar o código mais definido!

  • Manter sempre o mesmo idioma no código inteiro (evite misturar inglês e português), se preferir criar os nomes das variáveis e arquivos em português não é um problema, desde que mantenha a unidade no código inteiro.

  • Procure lembrar de fazer a documentação, no backend é de extrema importância!

No mais, parabéns demais pela sua dedicação!

Tenha uma excelente semana!

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