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(dynamic-view): Correção campos duplicados no dynamicview #2001

Closed
wants to merge 5 commits into from

Conversation

bruno-severino
Copy link
Contributor

**po-dynamic-view **

8512


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?
Ocorre uma duplicação de alguns campos mostrados no dynamicview, quando esses campos não possue valor ou eles estão com visibilidade falsa

Qual o novo comportamento?
correção feita, não ocorre mais a duplicata dos campos

Simulação

  • Abrir na documentação PO-UI > dynamic view. O Ultimo exemplo tem campos com visible false, que estava duplicando o campo 'city'
  • anexo
    simulação.zip
    Existe campo sem valores que estava duplicado o campo 'partner'

anderson-gregorio-totvs and others added 2 commits March 12, 2024 16:23
Ajusta a publicação na tag @beta do NPM

Fixes DTHFUI-8276
Solicita instalação das dependências obrigatórias para a migração

DTHFUI-8422
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch from 6038c5d to a74930c Compare March 14, 2024 19:32
integração via webhook com o discord do core team

DTHFUI-7986
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch from a74930c to 500b717 Compare March 18, 2024 14:20
@bruno-severino bruno-severino changed the base branch from master to development March 18, 2024 18:10
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch 2 times, most recently from f1b00df to 9dfc47e Compare March 18, 2024 19:37
@bruno-severino bruno-severino changed the title fix(dynamic-view) Correção campos duplicados no dynamicview fix(dynamic-view): Correção campos duplicados no dynamicview Mar 18, 2024
Base automatically changed from development to master March 18, 2024 21:42
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch from 9dfc47e to 0516914 Compare March 19, 2024 19:33
@bruno-severino bruno-severino changed the base branch from master to development March 19, 2024 19:37
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch from 0516914 to 07a5641 Compare March 19, 2024 19:39
Alterado local para adição do order para ignorar campos sem
valores ou sem visibilidade.

fixes DTHFUI-8512
@bruno-severino bruno-severino force-pushed the dynamicview/DTHFUI-8512 branch from 07a5641 to 0eaab6d Compare March 19, 2024 19:42
@bruno-severino bruno-severino deleted the dynamicview/DTHFUI-8512 branch March 19, 2024 21:27
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.

5 participants