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(po-page-dynamic-table): adiciona propriedade p-single-select #1723

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

PauloPenalva
Copy link

@PauloPenalva PauloPenalva commented May 24, 2023

Adiciona a propriedade p-single-select da po-table no componente po-page-dynamic-table

Fixes 1722

< po-page-dynamic-table >

< 1722 >


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?
Não temos a propriedade p-single-select da po-table na po-page-dynamic-table

<po-page-dynamic-table
p-auto-router
p-infinite-scroll
p-keep-filters
p-height="340"
p-single-select
p-title="Notas Fiscais de Entrada"
[p-service-api]="serviceApi"
[p-actions]="actions"
[p-page-custom-actions]="pageActions"
[p-breadcrumb]="breadcrumb"
[p-fields]="columns"

Qual o novo comportamento?
Adicionada a propriedade p-single-select da po-table na po-page-dynamic-table

<po-page-dynamic-table
p-single-select
p-auto-router
p-infinite-scroll
p-keep-filters
p-height="340"
p-single-select
p-title="Notas Fiscais de Entrada"
[p-service-api]="serviceApi"
[p-actions]="actions"
[p-page-custom-actions]="pageActions"
[p-breadcrumb]="breadcrumb"
[p-fields]="columns"

**Simulação** Comportamento padrão

image

Comportamento com p-single-select

image

Adiciona a propriedade p-single-select da po-table no componente
po-page-dynamic-table

Fixes 1722
@PauloPenalva PauloPenalva changed the title feat(table): adiciona propriedate p-single-select feat(po-page-dynamic-table): adiciona propriedade p-single-select May 24, 2023
@alinelariguet
Copy link
Member

@PauloPenalva muito obrigada pela contribuição! poderia ajustar o conflito da pull request? assim que ajustado vamos priorizar a revisão e liberação

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.

2 participants