-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
[14.0][IMP][l10n_br_cte_spec] CTe NT2024.002 - Publicado em 08/04/2024 #3127
base: 14.0
Are you sure you want to change the base?
Conversation
e5d4110
to
5272298
Compare
c7faea3
to
8e7ffd8
Compare
8e7ffd8
to
fad296b
Compare
o diff ficou um pouco grande por conta da mudança da versao do Black (futuramente irei usar Ruff como na OCA tb). Ai eu separei
Nisso o PR ficou pronto para ser revisado. cc @marcelsavegnago |
fad296b
to
742283a
Compare
@mileo consegue revisar ? |
talvez isso quebra os PRs de CTe que a Kmee botou uns meses atras (a NT entrou depois). Nisso talvez vale a pena já limpar um pouco os PRs da CTe existentes antes de talvez quebrar eles sem dominar muito com essa atualização dos esquemas. |
Vejo que é melhor priimeiro fazer o merde dos outros prs, a não ser que seja obrigatório a transmissão esse esquema novo. |
Você pode referenciar os PRs que você esta trabalhando para ficar relacionado com esse PR para ficar claro e melhor para esperar os merges dos outros PRs antes de fazer o merge desse. |
Pessoal, por mim mescla viu... vamos ter que mexer de um jeioto ou de outro e temos os commits das prs para poder ver oq foi alterado. |
ping @mileo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por mim é OK
De uma forma ou de outra vamos ter que dar uma trabalhada neste módulo e talvez segregar o CTEOS e outros modelos e deixar este apenas para o CT-e Normal... No momento tive que comentar todas as classes que nao estão relacionadas com o CTE e e renomear os campos onde foram adicionados prefixos para separar o campo com o mesmo nome do CTE do CTEOS. |
Não vale a pena deixar tudo do CTE-OS em outro módulo? |
ja era a ideia de separar o cteOS em outro módulo... |
assim no pacote de XSD da CTe, no mesmo arquivo temos as tags da CTe "TCte" https://github.com/akretion/nfelib/blob/master/nfelib/cte/schemas/v4_0/cteTiposBasico_v4.00.xsd#L2688 e as tags da CTeOS "TCTeOS" https://github.com/akretion/nfelib/blob/master/nfelib/cte/schemas/v4_0/cteTiposBasico_v4.00.xsd#L4947 logo na geração dos mixin Odoo ja tinha que lidar com a situação de diferenciar o que é mixin de CTe e o que é mixin de CTeOS. Quando eu gerei os mixins da MDFe e da CTe eu tive que fazer algumas adaptaçoes. Pois nos 2 tem um "polimorfismo" com o modo de transporte. Mas na CTe tem até essa questão da CTeOS que tem tags filhos com o mesmos nomes de que tags filhos da CTe e ai eu tive que adaptar alem ainda para a CTe. Pelo jeito atendeu bem a MDFe e até meus testes basicos com a CTe, mas pode ser que precisa ajustar mais alguma coisa... @marcelsavegnago seu feedback com isso esta muito bem vindo e me comprometo em ajudar a ajustar o gerador se precisar. |
@rvalyi o MDFe foi de boa.. agora o CTE como comentei, precisei comentar cteos e afins e tem algumas coisas que estão dando trabalho e por ora estou considerando que pode ser erro meu massssss pode ser alguma coisa do _spec. |
Atualizacao referente a atualizacao CTe NT2024.002 - Publicado em 08/04/2024 dos esquemas de CTe. Na nfelib eu tinha feito a atualizacao um tempo atras ja aqui: akretion/nfelib#99
Ja que tem pessoas mexendo na CTe de novo, vale a pena atualizar. Novamente sao coisas triviais de sincronizar com as branches mais novas depois.
Eu tambem aproveitei para corrigir o summary no l10n_br_mdfe_spec num outro commit.
cc @marcelsavegnago @renatonlima @mileo @antoniospneto @felipemotter