[12.0] l10n_br_nfe-ak-filtered-odb

[mileo]

Pessoal fiz esse PR, mas lembrando que tem o #1043

A ideia é ver se os testes estão verdes e já partir para o merge, deixando as correções em outros PRs.

Coloquei o módulo como:

    "development_status": "Beta",

@rvalyi @marcelsavegnago @renatonlima @gabrielcardoso21

[mileo]

Pessoal fiz mais uma pequena correção nos pagamentos nesse commit: a15bc77

[mileo]

@renatonlima @rvalyi se puderem revisar, fiz algumas pequenas melhorias.

[mileo]

@renatonlima @rvalyi se puderem revisar, fiz algumas pequenas melhorias.

Já esta com o rebase da 12.0.

[mileo]

Uma coisa que eu notei e esqueci de comentar é que não é possível inserir dados de pagamento através dos dados de demo.

Tentei de todo jeito melhorar esse trecho:

https://github.com/odoo-brazil/l10n-brazil/blob/12.0-l10n_br_nfe-ak-filtered-odb/l10n_br_nfe/demo/fiscal_document_demo.xml#L104-L115

Inclusive com diversas modificações e a unica forma foi colocar nesse trecho:

https://github.com/odoo-brazil/l10n-brazil/blob/12.0-l10n_br_nfe-ak-filtered-odb/l10n_br_nfe/tests/test_nfe_serialize.py#L54

Acredito que seja algum problema na ordem da inicialização do spec, provavelmente a relação só é estabelecida em um momento posterior.

cc @rvalyi

[marcelsavegnago]

@mileo legal.. ficou ótimo.

Inclusive ficou melhor o teste de serialização.. parabéns.

@rvalyi @renatonlima Conseguem dar uma olhada ?

[rvalyi]

pessoal, nada contra esses ultimos commits do meu lado. Porem a gente observou algumas regressões em prod com alguns commits de umas 3 semanas para ca. Estamos olhando isso para dar um retorno ainda.

[gabrielcardoso21]

pessoal, nada contra esses ultimos commits do meu lado. Porem a gente observou algumas regressões em prod com alguns commits de umas 3 semanas para ca. Estamos olhando isso para dar um retorno ainda.

Can you tell us what are those regressions? Just in case we need to apply a quick fix in our version before you come up with the definitive solution

[marcelsavegnago]

Bora fazer merge :D ????

[marcelsavegnago]

pessoal, nada contra esses ultimos commits do meu lado. Porem a gente observou algumas regressões em prod com alguns commits de umas 3 semanas para ca. Estamos olhando isso para dar um retorno ainda.

@rvalyi precisa de ajuda em algo ?

[rvalyi]

pessoal, nada contra esses ultimos commits do meu lado. Porem a gente observou algumas regressões em prod com alguns commits de umas 3 semanas para ca. Estamos olhando isso para dar um retorno ainda.

Can you tell us what are those regressions? Just in case we need to apply a quick fix in our version before you come up with the definitive solution

Looks like it comes from this PR https://github.com/akretion/l10n-brazil/pull/148/files, we have regressions when printing the DANFE after signing or for updating the document state after the transmission.

[rvalyi]

may be this commit specifically: akretion@de00615

[gabrielcardoso21]

Looks like it comes from this PR https://github.com/akretion/l10n-brazil/pull/148/files, we have regressions when printing the DANFE after signing or for updating the document state after the transmission.

Thank you for the heads up. We're going to look out for these problems and if we find a solution we'll make a pull request here.

[mileo]

may be this commit specifically: akretion@de00615

This is related to a issue of the new version of erpbrasil.edoc.pdf, fallback to erpbrasil.edoc.pdf==0.1.2

[mileo]

@rvalyi

[mileo]

Pessoal estou verificando o Erro.

[mileo]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-1272-by-mileo-bump-nobump, awaiting test results.

[OCA-git-bot]

@mileo your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1272-by-mileo-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[mileo]

Pessoal estou achando que o problema esta na tradução.

Reparem no histórico: https://github.com/OCA/l10n-brazil/commits/e3c7a009d7fae464bc9bd6ccc868e5cdb10b1927?before=e3c7a009d7fae464bc9bd6ccc868e5cdb10b1927+35&branch=e3c7a009d7fae464bc9bd6ccc868e5cdb10b1927

Por exemplo para o módulo l10n_br_nfse a tradução rodou 11x seguidas e de formas diferentes e s/ sentido. Se observarmos o mesmo MODELO podemos verificar que ele foi traduzido de formas diferentes repetidamente:

1. d04d277

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "ipitrib"
msgstr ""

2. 2addd12

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Grupo de informação do ICMSST devido para a UF de destino, nas operações\n"
"interestaduais de produtos que tiveram retenção antecipada de ICMS por\n"
"ST na UF do remetente. Repasse via Substituto Tributário."
msgstr ""

3. 3b54b0b

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Veículos novos"
msgstr ""

4. 65687e2

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Código de Situação Tributária do PIS.\n"
"99 - Outras Operações."
msgstr ""

5. a621eb3

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Dados do PIS Substituição Tributária"
msgstr ""

6. fd057da

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Dados dos detalhes da NF-eNúmero do item do NF"
msgstr ""

7. 53cdc89

#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Dados do COFINS da\n"
"Substituição Tributaria;"

8. 4b7b0fe

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Grupo de informação do ICMSST devido para a UF de destino, nas operações\n"
"interestaduais de produtos que tiveram retenção antecipada de ICMS por\n"
"ST na UF do remetente. Repasse via Substituto Tributário."
msgstr ""

9. c40146e

#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Tipo: Dados do IPI"

10. dced031

#. module: l10n_br_nfse
#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Código de Situação Tributária do COFINS:\n"
"04 - Operação Tributável - Tributação Monofásica - (Alíquota Zero);\n"
"06 - Operação Tributável - Alíquota Zero;\n"
"07 - Operação Isenta da contribuição;\n"
"08 - Operação Sem Incidência da contribuição;\n"
"09 - Operação com suspensão da contribuição;"
msgstr ""

11. 10591e1

#: model:ir.model,name:l10n_br_nfse.model_l10n_br_fiscal_document_line
msgid "Dados do Imposto de Importação"

[rvalyi]

Eu diria que a gente pode tentar rodar o merge manual. o bot de traduçao e os outros vao rodar depois de noite uma so vez (e a principio nao criar um loop desse). Eu acho remote a possibilidade de dar merda ja que esse é um merge correcto mesmo. A ideia é quebrar esse ciclo apenas. Agora se o merge der mais merda, a gente so tem que estar preparado a dar um force push para voltar como ta hoje e voltar a analisar a situaçao. O que voces acham?

[mileo]

Eu diria que a gente pode tentar rodar o merge manual. o bot de traduçao e os outros vao rodar depois de noite uma so vez (e a principio nao criar um loop desse). Eu acho remote a possibilidade de dar merda ja que esse é um merge correcto mesmo. A ideia é quebrar esse ciclo apenas. Agora se o merge der mais merda, a gente so tem que estar preparado a dar um force push para voltar como ta hoje e voltar a analisar a situaçao. O que voces acham?

Pelo que eu debuguei hj cedo esta relacionado com os modelos derivados no _force_stack_paths, mas precisei dar uma saída e só voltei agora. Vou tentar debugar um pouco mais para entender quando ocorre essa sobreposição dos description dos modelos.

Se em algumas horas não conseguir nada fácil e que não altere muita coisa comento aqui e fazemos o merge manual

[mileo]

@renatonlima @marcelsavegnago @rvalyi resolvido, se puderem revisar.

[mileo]

A descrição dos modelos nf-e foi mantida como esta:

E a descrição dos modelos fiscais tb:

Bastou dar um update no l10n_br_fiscal.

[mileo]

/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1272-by-mileo-bump-minor, awaiting test results.

[mileo]

Pode ser que traduções como essas causem problema:

f38f6a8

Uma possibilidade seria fazer com que todos os campos dos specs tenham translate=False.

[mileo]

Pode ser que traduções como essas causem problema:

f38f6a8

Uma possibilidade seria fazer com que todos os campos dos specs tenham translate=False.

Pelo andamento já vamos na terceira vez que o mesmo arquivo é traduzido.

https://github.com/OCA/l10n-brazil/commits/12.0-ocabot-merge-pr-1272-by-mileo-bump-minor

Pelo jeito o problema dos modelos foi resolvido, mas os dos campos ainda não.

[OCA-git-bot]

Congratulations, your PR was merged at 8dac1de. Thanks a lot for contributing to OCA. ❤️

[mileo]

Congratulations, your PR was merged at 8dac1de. Thanks a lot for contributing to OCA.

Aee! @gabrielcardoso21 @luismalta @rvalyi @renatonlima @marcelsavegnago cerveja por minha conta.

OBS: Podemos ter surpresas coma tradução dos campos que tem colisão por exemplo: nfe40_vBC