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

(refactor) Busca de disciplina mais robusta #148

Open
isaacmsl opened this issue Jan 12, 2024 · 6 comments · May be fixed by #182
Open

(refactor) Busca de disciplina mais robusta #148

isaacmsl opened this issue Jan 12, 2024 · 6 comments · May be fixed by #182

Comments

@isaacmsl
Copy link
Owner

Como é o trecho de código atualmente?

Veja o que faltou fazer na #147

O que é esperado

Uma busca mais robusta, mais flexível para novos campos no frontend.

@Pablo1Gustavo
Copy link

Acho que seria interessante filtrar de forma mais generica sem precisar de dezenas de campos como o SIGAA faz, não tenho muita experiencia no Spring mas talvez desse para manter a mesma estrutura. No caso aproveitaria o mesmo input dado no campo de pesquisa e ele servir para o nome e codigo. Algo como WHERE nome LIKE '%input%' OR codigo=input.

@isaacmsl
Copy link
Owner Author

isaacmsl commented Jan 12, 2024

É uma boa mesmo. Faz sentido pela facilidade e nosso frontend mostra as disciplinas com o código junto com o nome, dando uma sensação de poder de busca. Por fim, acho que o campo deva ter Nome ou código como placelhoder.

@Pablo1Gustavo
Copy link

Justo!

@isaacmsl
Copy link
Owner Author

isaacmsl commented Jan 12, 2024

Por que nomes t ão grandes?

@Pablo1Gustavo, você diz em relação aos nomes das disciplinas estarem mostrando muitos caracteres ou o tamanho da fonte?

@Pablo1Gustavo
Copy link

Foi aquela dúvida que já tirei com você. Era sobre a quantidade de caracteres mesmo.

@isaacmsl
Copy link
Owner Author

Esse é um problema importante de ser resolvido também. Criada a issue #152 .

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 a pull request may close this issue.

2 participants