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): ajusta renderização do multiselect #2310

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

wsteixeira
Copy link
Contributor

Ajusta renderização de campo com multiplas opções.

Fixes #2308

Dynamic-View

2308


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?
Componente não renderiza corretamente campo com múltiplas opções selecionadas.

Qual o novo comportamento?
Componente renderiza corretamente campo com múltiplas opções selecionadas.

Simulação
Pode ser realizada com o APP

@anabye
Copy link
Contributor

anabye commented Nov 25, 2024

Bom dia @wsteixeira

Agradecemos a contribuição.
A PR foi analisada e identificamos algumas ações necessárias para seguir com o fluxo:

1. Ajuste na descrição do commit:

De: fix(dynamic-form): ajusta renderização do multiselect
Para: fix(dynamic-view): ajusta renderização do multiselect

  • Pois a correção foi feita no componente dynamic-view

2. Abertura de uma nova PR
Peço também que abra uma PR adicional para 17.x.x (versão LTS), pois mantemos suporte LTS para essa versão em casos de correções de bugs.

Assim que os ajustes forem realizados e revisados, faremos a publicação da correção na próxima segunda-feira (02.12).

Ficamos à disposição e aguardamos as atualizações.

@wsteixeira
Copy link
Contributor Author

Obrigado @anabye pelo retorno, vou realizar os ajustes necessários!

@wsteixeira wsteixeira changed the title fix(dynamic-form): ajusta renderização do multiselect fix(dynamic-view): ajusta renderização do multiselect Dec 2, 2024
@wsteixeira
Copy link
Contributor Author

Olá @anabye,

Me desculpe a demora, mas na semana passada está fora em um treinamento. Eu já ajustei a descrição do commit, mas não sei com abrir um PR para a versão 17!

@wsteixeira
Copy link
Contributor Author

wsteixeira commented Dec 4, 2024

Olá @anabye,

Obrigado pelas orientações!

Foi criada a PR #2325 para a versão 17.x.x

Ajusta renderização de campo com multiplas opções.

Fixes po-ui#2308
@anabye anabye merged commit d77f819 into po-ui:master Dec 5, 2024
7 of 8 checks passed
@anabye
Copy link
Contributor

anabye commented Dec 10, 2024

Obrigada novamente pelas contribuições @wsteixeira. A publicação foi feita e as alterações encontram-se disponíveis nas versões 18.19.0 e 17.26.9.

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.

Dynamic-view: Erro na exibição do componente multiselect com múltiplas opções
3 participants