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

Add query to find subjects with name, department and code params #182

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

RamonJales
Copy link
Collaborator

Por favor, informe se a PR segue os requisitos obrigatórios:

  • Mesmo padrão de código do projeto
  • Arquivos alterados/adicionados seguem o padrão Camel Case
  • A PR está relacionada a uma ou mais issue
  • Relacionei todas as issues na seção de "Development" da PR
  • O código foi revisado uma vez ou mais

Motivação para a criação da PR
Veja #148.

O que foi feito

  • Adicionado Query do mongodb.repository.Query para fazer busca pelos campos code, department e name.

@RamonJales RamonJales linked an issue Jan 17, 2024 that may be closed by this pull request
@RamonJales RamonJales requested review from isaacmsl and Kywal January 17, 2024 17:46
@isaacmsl
Copy link
Owner

isaacmsl commented Jan 18, 2024

Primeiro, fique atento ao padrão de nome de branch.
A tua deveria ter sido feat/add-query-find-by-camps (aliás camps é bem estranho, o usual é fields).
Segundo por que o Github Action não foi acionado? 🤔 Acho que a branch foi aberta antes mesmo da action ser mergeada no projeto. Vou testar.

Copy link
Owner

@isaacmsl isaacmsl left a comment

Choose a reason for hiding this comment

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

Tá lindo demais. Isso com certeza refatora bastante coisa!
Só não acho suficiente para fechar a #148 , só retira a dependência do nome do método gerar a busca. Não retira a obrigatoriedade de ter que passar os três parâmetros de uma vez. Ou estou enganado?

@RamonJales
Copy link
Collaborator Author

RamonJales commented Jan 18, 2024

O pior que não havia testado isso. Irei fazer um test para vê se dá certo. Quando você fala em não obrigatoriedade, se refere em passar parâmetros nulos?

Pois com string vazias funciona.

@isaacmsl
Copy link
Owner

Quando você fala em não obrigatoriedade, se refere em passar parâmetros nulos?

Posso fazer:
/subjects?name=bla
/subjects?department=bla
/subjects?name=bla&department=bli
/subjects?componentID=oi
/subjects?department=bla&componentID=eae

Como eu quiser... Só ir acumulando o que eu quero buscar. Qualquer campo.
Veja #148

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

(refactor) Busca de disciplina mais robusta
2 participants