Skip to content

PR de correção#10

Open
bellacbs wants to merge 21 commits intocorrecao-projetofrom
master
Open

PR de correção#10
bellacbs wants to merge 21 commits intocorrecao-projetofrom
master

Conversation

@bellacbs
Copy link

Olá Galera, segue o PR de correção, não precisam fazer o merge

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Tabela de turmas contendo id, nome e modulo
Tabela de estudantes contendo id, nome, email, data_nasc e turma_id (FK)
Tabela de docentes contendo id, nome, email, data_nasc e turma_id (FK)
Tabela de hobbies e tabela de junção (estudante/hobby)
Tabela de especialidades e tabela de junção (docente/especialidade)
Endpoint de criação de turma funcionando
Endpoint de 'busca de turmas ativas' e/ou 'mudança de módulo de turma' funcionando
Endpoint de criação de estudante funcionando
Endpoint de 'busca por nome de estudante' e/ou 'mudança de turma de estudante' funcionando
Endpoint de criação de docente funcionando
Endpoint de 'busca por todas as pessoas docentes' e/ou 'mudança de turma de docente' funcionando
[Desafio] Busca por todas as pessoas (estudantes e docentes) de uma mesma turma -
[Desafio] Busca por todas as pessoas com o mesmo hobby (estudantes) -
[Desafio] Busca por todas as pessoas com a especialidade 'POO' (docentes) -
[Desafio] Busca por todas as pessoas com o mesmo signo (estudantes e docentes) -
Feedback do código Feito

Comentários da pessoa avaliadora

Ola Galera, parabéns pela entrega, deixarei umas observações aqui:

  • quando fizerem uma condicionar nesses casos de acordo com a lógica suas e pensando que o node roda de forma linear, quando fizer o if, não precisam do eles, pois se ele caiu na condição, ele já sai…. Exemplo: vocês colocaram
if(!nome){
            errorCode = 422;
            throw new Error("Preencha seu nome")
        }
        else if(!email){
            errorCode = 422;
            throw new Error("Preencha seu email")
        }

nesse caso poderiam colocar

if(!nome){
            errorCode = 422;
            throw new Error("Preencha seu nome")
  }
 if(!email){
            errorCode = 422;
            throw new Error("Preencha seu email")
  }
  • para o endpoint de buscar estudantes, uma sugestão seria fazer a condição que se o usuário tiver enviado o nome na query, buscar no banco de dados com where. E se não tiver enviado o nome, buscar todos os estudantes. Pois é o que mais acontece no mercado quando temos essas queries

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.

3 participants