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

feat(page-dynamic-search): melhora o disclaimer para campos booleanos #1636

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wsteixeira
Copy link
Contributor

@wsteixeira wsteixeira commented Feb 16, 2023

Melhora o disclaimer para campos booleanos, exibindo o atributo booleanTrue ou booleanFalse quando os mesmos forem diferentes de undefined ao invés do atributo value.

Fixes #1307

Page Dynamic Search

1307


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
Ao adicionar um campo do tipo booleano nos filtros da pesquisa avançada, selecioná-lo e pressionar o botão pesquisar, é exibido o disclaimer com o valor true ou false separado por ":". Exemplo: Nome do campo: true

Qual o novo comportamento?
Ao adicionar um campo do tipo booleano nos filtros da pesquisa avançada, selecioná-lo e pressionar o botão pesquisar, é exibido o disclaimer com o valor do booleanTrue ou booleanFalse separado por ":". Exemplo: Nome do campo: Sim

Simulação
Pode ser realizado a simulação neste App.

Melhora o disclaimer para campos booleanos, exibindo o atributo
`booleanTrue` ou `booleanFalse` quando os mesmos forem diferentes de
`undefined` ao invés do atributo `value`.

Fixes po-ui#1307
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.

PoPageDynamicSearch - Pesquisa Avançada - Campos boleanos
1 participant