Skip to content

[14.0][IMP] l10n_br_fiscal,nfe: CSRT #3778

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

Draft
wants to merge 4 commits into
base: 14.0
Choose a base branch
from

Conversation

corredato
Copy link
Contributor

No description provided.

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima, @rvalyi,
some modules you are maintaining are being modified, check this out!

@corredato corredato force-pushed the imp/14.0-nfe_csrt branch from 7d99dd4 to 046c165 Compare May 7, 2025 17:35
@corredato corredato force-pushed the imp/14.0-nfe_csrt branch from 046c165 to 03875a5 Compare May 7, 2025 17:48
@corredato corredato force-pushed the imp/14.0-nfe_csrt branch 2 times, most recently from b4e72c5 to 9bf5b61 Compare May 7, 2025 17:57
@DiegoParadeda
Copy link
Contributor

Parece legal, esse PR é importante pois a inclusão dos campos idCSRT e hashCSRT tornou-se obrigatória no PR

@corredato corredato requested a review from antoniospneto May 7, 2025 18:17
@corredato corredato changed the title [14.0][IMP] l10n_br_nfe: csrt [14.0][IMP] l10n_br_base,nfe: CSRT May 7, 2025
@antoniospneto
Copy link
Contributor

antoniospneto commented May 7, 2025

@DiegoParadeda e @corredato mas será que o módulo base é melhor lugar pra por esses campos? como tá relacionado ao sped fiscal não é melhor por no l10n_br_fiscal?

Edit: Pelo que li, na real pode ser no módulo nfe mesmo.

@rvalyi
Copy link
Member

rvalyi commented May 7, 2025

@DiegoParadeda e @corredato mas será que o módulo base é melhor lugar pra por esses campos? como tá relacioado ao sped fiscal não é melhor por no l10n_br_fiscal?

concordo com isso, não é legal no l10n_br_base.

@corredato corredato force-pushed the imp/14.0-nfe_csrt branch from 9bf5b61 to d42d6ec Compare May 8, 2025 15:47
@corredato corredato changed the title [14.0][IMP] l10n_br_base,nfe: CSRT [14.0][IMP] l10n_br_fiscal,nfe: CSRT May 8, 2025
Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

pra mim parece ok, valeu!

Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

@DiegoParadeda
Copy link
Contributor

Já explico as mudanças pessoal, precisei fazer um WIP

@corredato corredato closed this May 8, 2025
@corredato corredato reopened this May 8, 2025
@OCA-git-bot
Copy link
Contributor

Hi @renatonlima, @rvalyi,
some modules you are maintaining are being modified, check this out!

@corredato corredato marked this pull request as draft May 8, 2025 17:51
@DiegoParadeda
Copy link
Contributor

DiegoParadeda commented May 9, 2025

🔧 Campos da tag na NF-e:

  • CNPJ
  • xContato
  • email
  • fone
  • idCSRT
  • hashCSRT

Detalhes:

  • infRespTec.idCSRT precisa ser específico por empresa (duas empresas podem ter idCSRT diferentes mesmo que compartilhem o mesmo technical_support_id em res.company).
  • infRespTec.hashCSRT é calculado dinamicamente com base no tokenCSRT e na chave da nota.

Os demais campos da tag infRespTec continuam sendo preenchidos a partir do responsável técnico definido no cadastro da empresa (res.company.technical_support_id).

🔜 Próximos passos:

  • Definir o melhor local para o cálculo do hashCSRT (atualmente implementado em WIP no método _generate_key).
  • Criar o campo tokenCSRT no res.partner (ao lado do idCSRT).

@corredato

@marcelsavegnago
Copy link
Member

@DiegoParadeda ela fica ainda como rascunho ?

@DiegoParadeda
Copy link
Contributor

@marcelsavegnago sim, a solução atual ta bem feia, foi um hotfix. Vamos implementar esses próximos passos que comentei pra ficar legal antes voltar pra review

Copy link
Member

@renatonlima renatonlima left a comment

Choose a reason for hiding this comment

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

@DiegoParadeda,

O capitulo "2.2.10. Responsável Técnico" do manual da NF-e/NFC-e diz que o responsável técnico pode pedir o número CSRT (Código de Segurança do Responsável Técnico) para a Sefaz de cada UF onde for obrigatório, cada responsável técnico pode solicitar até 5 CSRT por UF os estados onde são obrigatório até o momento são AM, MT, PE, PR, SC e TO.

Então o CSRT é solicitado para a empresa responsável técnico (technical_support_id) e não para a empresa emitente da NFe (res.company).

Não faz sentido o campo hashCSRT estar na empresa (res.company) ou no responsável técnico, ele deve ser implementado no documento fiscal já que ele é gerado concatenando o CSRT com a chave de acesso da NF-e/NFC-e que está sendo emitida e aplicado o algoritmo SHA-1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants