Skip to content

RTC no CT-e#124

Closed
flavionootech wants to merge 3 commits intoHercules-NET:masterfrom
flavionootech:master
Closed

RTC no CT-e#124
flavionootech wants to merge 3 commits intoHercules-NET:masterfrom
flavionootech:master

Conversation

@flavionootech
Copy link
Contributor

Criado campos do IBS e CBS para o CT-e

@gustavo-gerenesi
Copy link

@flavionootech

Em primeiro lugar, obrigado pela contribuição.

Trabalho com o @marcosgerene e ele pediu para revisar este PR (tinha uma OS aberta para mim aqui para justamente implementar isso). É minha primeira experiência com documentos fiscais então vou levar um pouco mais de tempo nisso, mas tento rever ainda nesta semana.

Se algum outro moderador quiser revisar antes pela urgência, sem problemas.

Copy link

@gustavo-gerenesi gustavo-gerenesi left a comment

Choose a reason for hiding this comment

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

ref:
Nota Técnica CTe - Nota_Técnica_2025_001 - RTC_v1.10 - Publicada em 06/10/2025

Encontrei alguns pontos que precisam de atenção e fiquei com uma dúvida quanto à origem de parte da implementação, especialmente nas classes relacionadas ao modelo monofásico (ex.: gIBSCBSMono e similares).

Agradeço pela paciência e pelo trabalho no PR.

Choose a reason for hiding this comment

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

faltou o campo "indDoacao"

Choose a reason for hiding this comment

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

referencia: https://www.cte.fazenda.gov.br/portal/listaConteudo.aspx?tipoConteudo=Y0nErnoZpsg=
RTC 1.10

faltou o campo indDoacao, sendo ele 0-1 (opcional)

public int? indDoacao { get; set; }
public bool ShouldSerializeindDoacao() => indDoacao.HasValue;

Choose a reason for hiding this comment

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

faltou o campo "gEstornoCred" - sendo ela um grupo que contem o vIBSEstCred e vCBSEstCred

Choose a reason for hiding this comment

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

"gIBSCredPess" e "gCBSCredPess" foram removidos do gIBSCBS

https://www.cte.fazenda.gov.br/portal/listaConteudo.aspx?tipoConteudo=Y0nErnoZpsg=

RTC v1.10 06/10

Choose a reason for hiding this comment

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

removido

Choose a reason for hiding this comment

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

não encontrei na docs, poderia me falar a fonte disso ?
também procurei no fonte do acbr

Choose a reason for hiding this comment

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

não encontrei na docs

Choose a reason for hiding this comment

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

não encontrei na docs referente ao rtc v1.10

Choose a reason for hiding this comment

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

não encontrei na docs referente ao rtc v1.10

Choose a reason for hiding this comment

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

não encontrei na docs referente ao rtc v1.10

Choose a reason for hiding this comment

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

não encontrei na docs

@marcosgerene
Copy link
Collaborator

@flavionootech

Me fala se você mesmo pretende ajustar para a nova versão da NT ou se devo aproveitar o que você já fez e pedir ao @gustavo-gerenesi adaptar.

Atenciosamente,
Marcos

@flavionootech
Copy link
Contributor Author

Olá, posso estar ajustando sim, amanha ou quarta devo fazer isso, obrigado pela revisão

marcosgerene added a commit to marcosgerene/ZeusFiscal that referenced this pull request Nov 24, 2025
@marcosgerene
Copy link
Collaborator

@flavionootech
A exatamente 1h atrás peguei seu PR e fiz o processo... haha

de qualquer forma agradeço a contribuição e gostaria que, se possível revisasse o novo PR.

@marcosgerene marcosgerene mentioned this pull request Nov 24, 2025
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.

3 participants