Conversation
Configurado projeto.
…spectivos endpoints
parte de criar estudante, professor e turma criada, junto com seus re…
Implementado funcionalidade de troca de turma.
…ora. Também criei um script que roda uma função que cria as tabelas.
Feito refatoramento nos códigos das funcionalidades existentes até ag…
feito o primeiro desafio, pegar nome dos estudantes por turma
Resolvido bug no script de criar tabelas.
…lunos quanto para professores.
Implementado funcionalidade de pegar idade.
Refatorado funcionalidade de pegar estudantes por turma.
Implementado funcionalidade de pegar professor pela classe.
Implementado funcionalidade de pegar alunos por hobby.
Implementado funcionalidade de remover aluno de turma.
Implementado funcinalidade de remover estudante do curso.
IMplementado funcionalidade de remover professor da turma.
Implementado funcionalidade de alterar módulo da turma.
jvpalves
left a comment
There was a problem hiding this comment.
Oi, gente! O projeto de vocês foi considerado acima do esperado. Meus parabéns!
Toda a fundação do módulo 4 parece ter sido muito bem absorvida, e deixa vocês prontíssimos para o módulo 5. Parabéns especiais às funções auxiliares que foram criadas, foram realmente uma mão na roda para os tratamentos de dados que foram feitos. Uma dica é que usar o DayJS para lidar com datas talvez possa ajudar! Até o próximo projeto
| export const getAge = async (idUser: string, tableName: string) => { | ||
| const birthdate = await connection.raw(` | ||
| SELECT birthdate | ||
| FROM ${tableName} | ||
| WHERE id = "${idUser}" | ||
| `) |
There was a problem hiding this comment.
O cálculo da idade podia ser feito direto no banco, sem problemas
| FOREIGN KEY (class_id) REFERENCES Class(id) | ||
| ); | ||
| `) | ||
| console.log('Funfou') |
There was a problem hiding this comment.
eu ri desse funfou hahaha, mas é uma trocar para algo como "Tabelas criadas com sucesso"
| if (req.body.type_class === 'Noturna') { | ||
| if (!req.body.name.includes('-na-night')) { | ||
| throw new Error('Nomes de turmas noturnas precisam terminar com -na-night.') |
There was a problem hiding this comment.
sugestao: podia ser apenas um if, com &&
| if (error.message.includes('type_class')) { | ||
| res.status(422).send('Este tipo de turma não existe.') |
| try { | ||
| const id = Number(req.params.id) | ||
|
|
||
| if(isNaN(id) === true) throw new Error("Invalid Id.") |
There was a problem hiding this comment.
pode ser contraído para if(isNaN(id))
Observações sobre o projeto