Skip to content

feat: criação da rota signup#5

Open
windson-dev wants to merge 3 commits intodevelopmentfrom
feat-cria-rota-signup
Open

feat: criação da rota signup#5
windson-dev wants to merge 3 commits intodevelopmentfrom
feat-cria-rota-signup

Conversation

@windson-dev
Copy link
Collaborator

@windson-dev windson-dev commented Dec 11, 2022

Descrição

-adiciona de uma nova rota /signup.
-adiciona ESLINT no projeto.
-password agora necessita de pelo menos um caractere especial e um numero.

Setup

  • docker-compose up -d
  • docker exec -it service_api bash
  • npm install
  • npm run debug
  • possuir thunderclient, insomnia, postman, entre outros softwares de requisição http
  • rota: http://localhost/3000/signup

Exemplo de requisição

{
  "nickname": "seunickname",
  "email": "seuemail@email.com",
  "password": "@suasenha123",
}

Caso de sucesso

  • Retorno de usuario criado com sucesso.
{
  "token": "TOKEN JWT"
}

Casos de falha

  • Retorno caso password não possua caracter especial e numero`
{
    "message": "\"password\" required one special character and one number"
}
  • Retorno caso usuario já exista no banco de dados
{
  "message": "Usuario já existe!"
}
  • Retorno caso password tenha menos de 6 caracteres
{
  "message": "\"password\" length must be 6 characters long"
}
  • Retorno caso nickname tenha menos de 8 caracteres
{
 "message": "\"nickname\" length must be at least 8 characters long"
}

@@ -0,0 +1,18 @@
const userService = require("../services/userService");

const isValidExistUser = async (req, res, next) => {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse middleware poderia ser apenas uma função que é chamada dentro do nosso service que vai lidar com as regras de negocio

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não entendi muito bem o que é pra ser feito

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