Skip to content

Cookenu#63

Open
Jucelia11 wants to merge 8 commits into
masterfrom
cookenu
Open

Cookenu#63
Jucelia11 wants to merge 8 commits into
masterfrom
cookenu

Conversation

@Jucelia11
Copy link
Copy Markdown
Collaborator

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

  • DESCREVA O QUE FUNCIONA NO SEU PROJETO

O que não funciona

  • DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

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

@pedrohenriqueduarte pedrohenriqueduarte 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ários contendo, no mínimo: id, nome, email e senha
Tabela de receitas contendo id, título, descrição, data de criação e id do autor
Requisição válida de cadastro é salva no banco e devolve um jwt
Requisição válida de login devolve um jwt
Requisição de buscar o próprio perfil devolve as informações corretas
Requisição de buscar perfil por id devolve as informações corretas
Requisição válida de criar receita é salva no banco
Requisição de buscar receita por id devolve as informações corretas
Requisições inválidas devolvem respostas de erro claras
Fez ao menos um desafio
Fez dois ou três desafios
Fez todos os desafios
Script para criar e popular as tabelas usando o Knex
Documentação da API
Feedback do código Feito
Classes para representar usuários e receitas
Classe para representar os serviços de autenticação
Nomes adequados para as variáveis e rotas

Comentários da pessoa avaliadora

Olá Jucelia, parabéns pela entrega!

Seu projeto ficou bem completo e todos os endpoints funcionando perfeitamente! Você optou por fazer num esquema sem arquitetura em camadas e não tem problema nisso. Mas acho que vale pena pensar no projeto como um todo em um paradigma só! Você poderia criar um controller para colocar as funções de endpoint lá.

No mais, ficou tudo certinho! Parabéns!

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