Skip to content

PR de Correção#11

Open
Asoaresdev wants to merge 42 commits intocorrecao-projetofrom
main
Open

PR de Correção#11
Asoaresdev wants to merge 42 commits intocorrecao-projetofrom
main

Conversation

@Asoaresdev
Copy link

PR de correção. Favor não dar merge

Copy link
Author

@Asoaresdev Asoaresdev 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 id, nome, email, senha e role
Tabela de bandas contendo id, nome, gênero musical e músico responsável
Tabela de shows contendo id, dia da semana, horários de início e término e id da banda
Requisição válida de cadastro é salva no banco e devolve um jwt
Requisição válida de login devolve um jwt
Requisição válida de registrar banda é salva no banco - ✅
Requisição de buscar banda por id devolve as informações corretas
Requisição válida de adicionar show a um dia é salva no banco -
Requisição de buscar shows por dia devolve as informações corretas
Requisições inválidas devolvem respostas de erro claras
Fez ao menos um desafio -
Fez mais de um desafio -
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 as entidades
Classes para representar os serviços de autenticação
Aplicou alguma arquitetura ao código
Nomes adequados para as variáveis e rotas
Testes para o endpoint de registrar banda (mínimo 2) -
Testes para o endpoint de adicionar show a um dia (mínimo 2) -

Comentários da pessoa avaliadora

Parabéns pela entrega, Caroline.
Seu projeto está muito bom! Realizei os testes nos endpoints e encontrei problema em dois que já relatei pessoalmente para você[RESOLVIDO]. Todos os outros funcionaram dentro do esperado e em caso de erro me lançava mensagem de erro clara.
Uma dica: Mesmo não estando no escopo do projeto, é bom ter endpoint que retorna as informações básicas da API, nesse caso, como banda, usuário e show. Precisei de id para testar endpoint e não tinha aonde buscar id, sorte que na documentação já havia um id válido
Seu código segui bem a proposta de arquitetura, ficando bem organizado.
Mais uma vez parabéns pelo projeto e pelo empenho na implementação do último projeto do curso!🎉🚀

@Asoaresdev
Copy link
Author

Sempre aprimorando o projeto! Parabéns!

@CarolineMartins09 CarolineMartins09 requested review from CarolineMartins09 and removed request for CarolineMartins09 August 9, 2023 14:31
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