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.

Sign up for GitHub

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

Jump to bottom

fix(dynamic-form): ajusta evento validate com método clean #1860

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(dynamic-form): ajusta evento validate com método clean #1860

wants to merge 1 commit into from

Conversation

wsteixeira
Copy link
Contributor

@wsteixeira wsteixeira commented Oct 26, 2023 •
edited by jnrpalma

O evento validate apenas é disparado se o usuário tocar no campo antes de clicar no X do clean.

Foi feito um ajuste para que o evento validate seja disparado mesmo o usuário clicar diretamente no X do clean sem tocar no campo.

Fixes #1837

Dynamic Form

1837

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?
O evento validate apenas é disparado se o usuário tocar no campo antes de clicar no X do clean.

Qual o novo comportamento?
Foi feito um ajuste para que o evento validate seja disparado mesmo o usuário clicar diretamente no X do clean sem tocar no campo.

Simulação

A simulação pode ser feita pelo APP

@wsteixeira
Copy link
Contributor Author

olá,

Vou tentar novamente, para ver se não é o CI.

@wsteixeira
fix(dynamic-form): ajusta evento validate com método clean
5473d66 
O evento `validate` apenas é disparado se o usuário tocar no campo
antes de clicar no `X` do `clean`.

Foi feito um ajuste para que o evento `validate` seja disparado mesmo
se o usuário clicar diretamente no `X` do `clean` sem tocar no campo.

Fixes po-ui#1837
@alinelariguet alinelariguet added the hacktoberfest-accepted Aceito para a hacktoberfest label Oct 31, 2023
@jnrpalma jnrpalma self-requested a review May 9, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jnrpalma jnrpalma jnrpalma approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
hacktoberfest hacktoberfest-accepted Aceito para a hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Dynamic Form: O evento validate não é disparado se o campo não for tocado
4 participants
@wsteixeira @jnrpalma @rafaellmarques @alinelariguet