Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Labook #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Labook #60

wants to merge 1 commit into from

Conversation

mhsouza88
Copy link
Collaborator

Labook

O que funciona

  • Nada ainda, o projeto foi iniciado.

O que não funciona

  • Tudo. 😅

Copy link

@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
Tabelas de usuários e posts
Requisições válida de cadastro e login devolvem um jwt
Requisições válidas de cadastro e criação de post são salvas no banco -
Requisição de buscar post por id devolve as informações corretas -
Requisições inválidas devolvem respostas de erro claras
Fez ao menos um desafio -
Fez quatro ou mais desafios -
Fez oito ou mais desafios -
Feedback do código Feito
Isolou as entidades da aplicação (usuários e posts)
Isolou os serviços de autenticação
Isolou as configurações e queries para o banco de dados
Isolou as configurações do servidor das lógicas de negócio
Utilizou corretamente as sintaxes de classes
Utilizou corretamente os routers do Express

Análise realizada por: Gabriel Mina

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