Skip to content

Conversation

@CaioVieira-dev
Copy link

card criar FormTable/FormList no RBU

Descrição

Este PR tem a inteção de implementar novos campos para os formularios do RBU.
Os novos campos são tabelas de itens. A ideia surgiu no https://github.com/geolaborapp/geolabor/pull/9241/files#diff-9113f84bb39b98160a16c4b5b9a9c2953784bb59b13c312af2303293d864e801, onde foi necesario um campo desses, e tivemos que fazer uma validação usando um throw new Error.

A ideia dos novos campo é criar uma maneira mais facil de implementar campos de "array" em formularios, e que esses campos possam ser validados pelo proprio form do RBU, sem a necessidade de validações no onSubmit

image image

sobre o uso do componente

As props mais importantes do componente são a "tableProps", a "getRemoveComponent" e a "getAddItemComponent".
Na "tableProps" é importante definir as columns e o actionLabel.
No "getRemoveComponent" a ideia é passar uma função que retorne um componente que vai lidar com a exclusão do item da tabela.
No "getAddItemComponent" a ideia é passar uma função que retorne um componente que vai gerar novos itens. Acredito que o mais comum vai ser um modal com outro form.

O spread do tableProps acontece depois das definições das props padrão que usei. Fiz isso para permitir customizar ainda mais o componente.

Ao revisor

Eu precisei fazer uma correção nas validações do UncontrolledForm. Do jeito que estava antes, as validações aparentemente só aconteciam no momento em que os campo era registrados. Agora elas devem acontecer sempre que o form sofrer alterações.

Ao tester

teste os novos campo nos dois forms

@CaioVieira-dev CaioVieira-dev added the aguardando primeira revisão Pull Request está aguardando a primeira revisão label Sep 26, 2025
Comment on lines +123 to +126

if (validations) {
this.validateForm();
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

essa foi a correção das validações no uncontrolled form

@nerissa-aguirre nerissa-aguirre added aguardando mesclagem PR está aguardando mesclagem and removed aguardando primeira revisão Pull Request está aguardando a primeira revisão labels Oct 17, 2025
@CaioVieira-dev CaioVieira-dev merged commit 39eb362 into master Oct 17, 2025
1 check passed
@CaioVieira-dev CaioVieira-dev deleted the adiciona-formtable branch October 17, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aguardando mesclagem PR está aguardando mesclagem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants