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

Feature/lider pesquisa colaborador #36

Merged
merged 8 commits into from
Jan 25, 2024

Conversation

mateuscavedini
Copy link
Contributor

Este PR resolve a issue #28.

Foi implementada a funcionalidade de busca por perfis para que o líder de um projeto possa futuramente convidá-los para fazer parte do mesmo.

Os dados dos perfis são resultado de uma requisição para uma API do Portifoliorrr.

Foi implementado um botão na tela de detalhes de um projeto que leva para a página de busca.

Como no momento a API da Portifoliorrr ainda não foi implementada, seguimos a ideia de um único endpoint, com um parâmetro opcional para a filtragem dos perfis.

Ao acessar a página, uma requisição é disparada para API, que retorna todos os usuários da Portifoliorrr. Com o envio de um termo, a API filtra os perfis pelo Tipo de Serviço.

Tela de detalhes, onde o botão para pesquisa foi implementado:
image

Tela com todos os perfis + campo de busca:
image

Tela com perfis filtrados:
image

Tela sem perfis a serem exibidos:
image

Débitos:

  • Adaptar o código para lidar com um array de job_category ao invés de uma string. Modificar assim que a API do Portifoliorrr ficar pronta.
  • Adicionar link para perfil completo/detalhes.

oLucasAguilar and others added 6 commits January 23, 2024 17:41
- Adiciona cenários de autenticação e autorização na pesquisa
- adiciona gem faraday

Co-authored-by: Mateus <[email protected]>
- Adiciona a validação para bloquear o botão da pesquisa

Co-authored-by: Mateus <[email protected]>
- Adiciona tratamento caso retorne erro interno na pesquisa

Co-authored-by: Mateus <[email protected]>
- Altera o nome global de contributors para portifoliorrr_profiles
- Adiciona teste de falha ao conectar no endpoint da API do Portifoliorrr

Co-authored-by: Mateus <[email protected]>
- Adiciona teste de um novo médodo personalizado de project
- Move testes de projeto para uma pasta própria

Co-authored-by: Mateus <[email protected]>
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.

A primeira versão me parece bastante promissora. Fiz uns comenários de melhorias iniciais que podem ser feitas :)

app/models/portifoliorrr_profile.rb Outdated Show resolved Hide resolved
app/views/portifoliorrr_profiles/search.html.erb Outdated Show resolved Hide resolved
- Refatora a condicional em caso de retorno de falha da API Portifoliorrr
- Refatora linha de código que era desnecessária no teste de busca

Co-authored-by: Mateus <[email protected]>
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.

Bora time 🚀

@mateuscavedini mateuscavedini merged commit bfcced6 into main Jan 25, 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.

[Pesquisa] Líder de projeto pesquisa colaborador
3 participants