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

Cria endpoint para solicitação de participação em projetos #85

Merged
merged 11 commits into from
Feb 7, 2024

Conversation

fredericomozzato
Copy link
Contributor

@fredericomozzato fredericomozzato commented Feb 6, 2024

Alcançado com este PR

Este PR resolve a issue #76

Foi criado um endpoint que recebe requisições POST e cria uma nova solicitação no banco de dados. que fica pendente até o líder do projeto aceitar ou recusar.

A requisição parte da aplicação Portfoliorrr e necessita dos seguintes campos:

  • ID do projeto (obrigatório)
  • ID do perfil do usuário que fez a requisição (obrigatório)
  • E-mail do usuário (obrigatório)
  • Mensagem (opcional)

Quando uma solicitação é criada com sucesso o endpoint retorna o ID da solicitação dentro do atributo data, conforme acordado entre os times.

Validações

Foram implementadas validações para garantir que uma solicitação somente será criada caso o usuário que a fez não seja membro do projeto que foi solicitada a entrada, ou caso já exista uma solicitação pendente para este projeto pelo mesmo usuário.

Documentação

O novo endopoint foi documentado junto com seus comportamentos de erro na documentação da API:

image

fredericomozzato and others added 9 commits February 5, 2024 17:55
Co-authored-by: lucasaguilar835@hotmail.com <lucasaguilar835@hotmail.com>
Co-authored-by: lucasaguilar835@hotmail.com <lucasaguilar835@hotmail.com>
Garante que uma solicitação precisa ter um id de perfil válido e se referir a um projeto existente na plataforma

Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
…ual fez a solicitação

Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: lucasaguilar835@hotmail.com <lucasaguilar835@hotmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Copy link
Contributor

@joaorsalmeida joaorsalmeida left a comment

Choose a reason for hiding this comment

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

Muito bom turma !

Copy link
Contributor

@joaorsalmeida joaorsalmeida left a comment

Choose a reason for hiding this comment

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

Tem um erro na documentação: ao invés de POST está escrito que o endpoint é um GET

Copy link
Contributor

@valerialrc valerialrc left a comment

Choose a reason for hiding this comment

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

Bom trabalho!

Co-authored-by: Lucas Aguilar <lucasaguilar835@gmail.com>
Copy link
Contributor

@decoporteira decoporteira left a comment

Choose a reason for hiding this comment

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

Muito bom!

Copy link
Contributor

@mateuscavedini mateuscavedini left a comment

Choose a reason for hiding this comment

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

Bom trabalho, pessoal! Código muito bem escrito! :)

Co-authored-by: Lucas GO Aguilar <olucasaguilar@users.noreply.github.com>
@OtavioLins OtavioLins dismissed joaorsalmeida’s stale review February 7, 2024 17:54

O solicitado já foi atendido

@oLucasAguilar oLucasAguilar merged commit f655429 into main Feb 7, 2024
2 checks passed
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.

[API] Solicitação de participação em projeto
6 participants