-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 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]>
Co-authored-by: Mateus <[email protected]>
joaorsalmeida
requested changes
Jan 24, 2024
There was a problem hiding this 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 :)
- 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]>
joaorsalmeida
requested changes
Jan 25, 2024
Co-authored-by: Mateus <[email protected]>
joaorsalmeida
approved these changes
Jan 25, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bora time 🚀
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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:
Tela com todos os perfis + campo de busca:
Tela com perfis filtrados:
Tela sem perfis a serem exibidos:
Débitos:
job_category
ao invés de uma string. Modificar assim que a API do Portifoliorrr ficar pronta.