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

fix(lookup): resolve disclaimer do lookup para o filtro booleano #1794

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

CSimoesJr
Copy link
Contributor

@CSimoesJr CSimoesJr commented Aug 24, 2023

PO-LOOKUP-MODAL

fixes DTHFUI-7511


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?
Após realizar algum filtro na busca avançada do lookup, o resultado no disclaimer é apenas o valor e se for do tipo booleano aparece o valor true ou false ao invés das propriedades 'booleanTrue' ou 'booleanFalse'. Exemplo: O campo do tipo booleano com o nome de 'Tem carro' com os valores do switch sendo 'Sim' para true e 'Não' para false aparece no disclaimer quando filtrado somente 'true'

Qual o novo comportamento?
Após realizar algum filtro na busca avançada do lookup, o resultado no disclaimer é a label concatenada com o valor e se for do tipo booleano aparece o valor das propriedades 'booleanTrue' ou 'booleanFalse'. Exemplo: O campo do tipo booleano com o nome de 'Tem carro' com os valores do switch sendo 'Sim' para true e 'Não' para false aparece no disclaimer quando filtrado 'Tem carro: Sim'.

Simulação
app.zip

jnrpalma
jnrpalma previously approved these changes Aug 25, 2023
jcorrea97
jcorrea97 previously approved these changes Aug 28, 2023
@CSimoesJr CSimoesJr dismissed stale reviews from jcorrea97 and jnrpalma via e747c28 August 29, 2023 19:21
Ajusta label do filtro booleano no disclaimer do lookup

fixes DTHFUI-7511
@rafaellmarques rafaellmarques merged commit 17e83c3 into master Aug 29, 2023
7 checks passed
@rafaellmarques rafaellmarques deleted the lookup/DTHFUI-7511 branch August 29, 2023 20:24
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.

4 participants