Skip to content

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.

@felipepetuco
Copy link
Contributor

Olá @wsteixeira atualize sua branch por gentileza, para que eu possa analisar a sua sugestão.

@felipepetuco felipepetuco added the status: awaiting author Additional information has been requested from the author label Aug 29, 2024
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
@wsteixeira wsteixeira force-pushed the page-dynamic-search/1307 branch from 598a3c9 to c087716 Compare September 2, 2024 12:44
@wsteixeira
Copy link
Contributor Author

Olá @felipepetuco, branch atualizada!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution status: awaiting author Additional information has been requested from the author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PoPageDynamicSearch - Pesquisa Avançada - Campos boleanos

3 participants