Skip to content

feat(nfse): confirm existing tax fiscal associations per company (#431) - #488

Merged
vitormattos merged 3 commits into
mainfrom
feat/tax-association-admin-431-20261014
Oct 14, 2026
Merged

vitormattos merged 3 commits into
mainfrom
feat/tax-association-admin-431-20261014

Conversation

@vitormattos

@vitormattos vitormattos commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Entrega

Tela compacta na configuração NFS-e para revisar e confirmar impostos Akaunting existentes, sem cadastro automático, mudança de alíquota, motor tributário ou novo onboarding. Reaproveita TaxFiscalAssociationStore e as enumerações fiscais da #428.

  • GET e POST com ACL distintas; dados e tax_id vinculados à empresa ativa.
  • Identidade, papel, fonte, vigência e autor explicitamente confirmados.
  • Reenvio idêntico idempotente, com validação existente de sobreposição e segurança de tenancy.
  • Estado desconhecido e compatibilidade contábil não são tratados como prova de incidência.
  • Testes Feature: isolamento, tentativas cruzadas, idempotência, não alteração dos impostos, sobreposição e confirmação explícita.
  • Documentação de operador e link nativo para impostos do Akaunting.

Sem alterações no Core, nas notas emitidas ou nos cálculos da #429. Commit assinado com DCO verificado. Refs #431 e #426.

CI deve aprovar antes de merge; registrar captura de tela com dados sintéticos na revisão visual.

Falha intermitente identificada na integração

A CI de integração com a main apresentou falha em CompetenceClosingReportTest::testClosingAndCsvExcludeForeignCompanyFiscalDocuments. A falha foi reproduzida com chaves sintéticas iguais por colisão do gerador de fixtures (random_int(1, 8) repetido 50 vezes); não demonstrou vazamento fiscal. A correção isolada e determinística está no PR #494, cuja CI passou. Revalidar os checks de integração desta branch depois que #494 for integrado. Não aumentar timeouts, ignorar asserções nem aprovar este PR com teste falhando.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
…ofiles

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos force-pushed the feat/tax-association-admin-431-20261014 branch from eef3679 to 6f1aef2 Compare October 14, 2026 19:09
@vitormattos
vitormattos merged commit 03a1d48 into main Oct 14, 2026
18 checks passed
@vitormattos
vitormattos deleted the feat/tax-association-admin-431-20261014 branch October 14, 2026 19:17
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.

1 participant