Tudo meio estranho#4
Open
Luzum wants to merge 1 commit into
Open
Conversation
labenu-bot
approved these changes
Aug 5, 2021
Contributor
labenu-bot
left a comment
labenu-bot
left a comment
There was a problem hiding this comment.
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 Gente, parabéns pelo trabalho! O projeto atende algumas das principais funcionalidades com um código bem estruturado e organizado. È hora de comemorar com o time, trocar elogios e agradecer pela colaboração. Abaixo farei algumas observações no código.
Análise realizada por: Índio
indiomedeiros
approved these changes
Aug 5, 2021
Comment on lines
+9
to
+17
| await connection ('class_labenu').insert ({ | ||
| id: 1, | ||
| name, | ||
| start_date, | ||
| end_date, | ||
| module | ||
| }) | ||
|
|
||
|
|
There was a problem hiding this comment.
O id precisa ser único, por enquanto podem até usar o Date.now().toString() ou o lib uuid https://www.npmjs.com/package/uuid
Comment on lines
+7
to
+20
| try {connection.raw(` | ||
| CREATE TABLE class_labenu( | ||
| id INT PRIMARY KEY NOT NULL, | ||
| name VARCHAR(255) NOT NULL, | ||
| start_date DATE NOT NULL, | ||
| end_date DATE NOT NULL, | ||
| module ENUM ('0','1','2','3','4','5','6','7') DEFAULT '0' | ||
|
|
||
| ); | ||
| CREATE TABLE student_labenu( | ||
| id INT PRIMARY KEY NOT NULL, | ||
| name VARCHAR(255) NOT NULL, | ||
| email VARCHAR(255) NOT NULL, | ||
| birth_date DATE NOT NULL, |
There was a problem hiding this comment.
Faltou o await antes do connection.raw await connection.raw()
Comment on lines
+8
to
+12
|
|
||
| const {name, start_date, end_date, module} = req.body | ||
| await createClass( | ||
| req.body.id, | ||
| req.body.name, | ||
| req.body.start_date, | ||
| req.body.end_date | ||
|
|
||
| name, | ||
| start_date, |
There was a problem hiding this comment.
È importante fazer as validações antes de tentar inserir os dados no Banco:
dica: pode usar o thow new Error("nome do erro")
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
O que não funciona
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