Skip to content

feat: torna CEP obrigatório + fix indicador de cadastro incompleto respeita hiddenFields - #42

Merged
ezequias21 merged 2 commits into
mainfrom
feature/citizen-form-required-cep
Sep 11, 2026
Merged

ezequias21 merged 2 commits into
mainfrom
feature/citizen-form-required-cep

Conversation

@lucasn4s

Copy link
Copy Markdown
Contributor

O que muda

1. CEP obrigatório no formulário (feat)

  • citizenFormFields.ts: campo CEP agora required: true com rules: 'required'
  • Snapshot do CitizenForm.test.ts atualizado

2. Fix: indicador de cadastro incompleto respeita hiddenFields no endereço (fix)

  • Address.isIncompleteExcept(): nova checagem que ignora campos listados em hiddenFields
  • Citizen.getContactInfo(): usa isIncompleteExcept no item Endereço em vez de isIncomplete (sem o fix, o alerta de endereço incompleto aparecia mesmo com o campo escondido)
  • CitizenSummaryViewer: lista de campos do endereço passa a incluir cep (sem isso, CEP faltando não acionava o aviso "Cadastro incompleto")

Testes

  • 78 passando / 7 skipped — incluindo 4 novos casos (CEP faltando escondido/não-escondido no viewer e no getContactInfo)

Release

Bump automático via semantic-release (commit feat: → minor, fix: → patch).

lucasn4s and others added 2 commits September 11, 2026 16:18
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Adiciona Address.isIncompleteExcept() para checar completude de endereço ignorando campos ocultos. O getContactInfo agora usa esse método ao invés de isIncomplete, e o CitizenSummaryViewer adiciona 'cep' na lista de campos do endereço.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@ezequias21
ezequias21 merged commit dd30430 into main Sep 11, 2026
5 checks passed
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