Skip to content

Conversation

@JULLIAIP
Copy link
Collaborator

No description provided.

@JULLIAIP JULLIAIP requested a review from SilvioDiaz July 30, 2021 20:21
Copy link
Contributor

@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
Tabela de estudantes contendo id, nome, email, data de nascimento e id da turma
Tabela de docentes contendo id, nome, email, data de nascimento e id da turma (pode estar unificada com a de estudantes)
Tabela de turmas contendo id, nome, tipo, módulo e datas de início e término
Tabela de hobbies contendo id do estudante e id (ou nome) do hobby
Tabela de especialidades contendo id do docente e id (ou nome) da especialidade -
Ao menos uma tabela de junção
Requisição válida de criar estudante é salva no banco
Requisição válida de criar docente é salva no banco
Requisição válida de criar turma é salva no banco
Requisição válida de adicionar estudante a turma é salva no banco -
Requisição válida de adicionar docente a turma é salva no banco -
Requisição válida de buscar estudante por id retorna a idade correta -
Requisições inválidas devolvem respostas de erro claras -
Fez ao menos um desafio
Fez três ou mais desafios -
Fez todos os desafios -
Script para criar e popular as tabelas usando o Knex -
Documentação da API -
Feedback do código Feito
Criou variáveis de tipo para representar as entidades
Nomes adequados para as variáveis e rotas -

Comentários da pessoa avaliadora

Oi Pessoal! O projeto de vocês atendeu algumas das funcionalidades esperadas. Abaixo deixarei comentários no código.

Análise realizada por: Índio

Comment on lines +9 to +18
try {
const result:any = await connection ("estudantes")
.select("*")

res.send(result[0])

} catch (error) {
return

}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quando usar o Query Builder não precisa do result**[0]**
É importante monitorar os erros para conseguimos entender o que está ocorrendo com a aplicação.
res.status(400).send({error: error.msqlMessage || error.message})

@SilvioDiaz SilvioDiaz removed their request for review December 17, 2021 21:32
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.

4 participants